Skip to content

Add directional experiment archiving in CLI and dashboard - #451

Merged
sox8502 merged 9 commits into
mainfrom
daniel/experiment-archive
Sep 28, 2026
Merged

sox8502 merged 9 commits into
mainfrom
daniel/experiment-archive

Conversation

@sox8502

@sox8502 sox8502 commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add exclusive archive and restore scopes for ancestors, the selected experiment, and descendants in both the CLI and dashboard.
  • Hide archived experiments in the graph behind a compact region marker with a direct restore action; put Show archived in the experiment filter.
  • Preserve archived experiments, runs, logs, and code for inspection.

Checks

  • cargo fmt --all --check, cargo clippy --all-targets -- -D warnings, cargo build --locked, cargo test --locked
  • pnpm build in ui/; regenerated ui/dist
  • Tried all three graph archive scopes, region restore, archived visibility, and archived-node actions in the isolated dev slot
  • Verified CLI help requires exactly one scope and rejects the removed --down option
  • User functionality check on the final reviewed revision

@greptile-apps

greptile-apps Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Adds experiment archiving feature to CLI and dashboard.

The PR appears safe to merge based on the reviewed changes.

Summary

The PR adds directional experiment archiving and restoration to the CLI and dashboard, while keeping archived data available for inspection.

  • Archived graph regions collapse into restoreable pills, and the experiment filter controls archived visibility.
  • The latest revision limits task-scoped pill restoration to experiments from the represented task and broadens accepted arXiv citation forms.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Experiment tree] --> B{Show archived?}
  B -- Yes --> C[Show experiment cards]
  B -- No --> D[Collapse contiguous archived region into pill]
  D --> E{Scope}
  E -- Current task --> F[Restore task region]
  E -- Entire project --> G[Restore project region]
Loading

Reviews (6) · Last reviewed commit: "Merge remote-tracking branch 'origin/mai..."

Comment thread ui/src/App.tsx Outdated
Comment thread ui/src/components/ArchiveMenu.tsx Outdated
@sox8502

sox8502 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

Comment thread ui/src/App.tsx Outdated
@sox8502

sox8502 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

…hive

# Conflicts:
#	src/plane/local_plane.rs
#	ui/dist/assets/PdfPreview-DsGiNUWo.js
#	ui/dist/assets/PdfPreview-MSqPpTd2.js
#	ui/dist/assets/PdfPreview-zPnQv7In.js
#	ui/dist/assets/index-BBHmyqxa.js
#	ui/dist/assets/index-BPWMgrXw.js
#	ui/dist/assets/index-DGMb5Jdi.js
#	ui/dist/index.html
@sox8502

sox8502 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

Comment thread ui/src/components/TreeView.tsx
@sox8502

sox8502 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@sox8502

sox8502 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@sox8502
sox8502 merged commit 169aad2 into main Sep 28, 2026
17 checks passed
@myles332 myles332 mentioned this pull request Sep 28, 2026
2 of 4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant