experimental/air: hyperlink Run ID/Experiment, MLflow run name, and i/L in air list - #6260
Open
riddhibhagwat-db wants to merge 1 commit into
Open
experimental/air: hyperlink Run ID/Experiment, MLflow run name, and i/L in air list#6260riddhibhagwat-db wants to merge 1 commit into
air list#6260riddhibhagwat-db wants to merge 1 commit into
Conversation
…add i/L in `air list` Enriches the interactive `air list` picker toward the Python CLI: - Run ID and Experiment cells are now OSC-8 hyperlinks (job-run page and MLflow experiment page, org-pinned), alongside the existing MLflow link. Underlined only when actually linked; piped/NO_COLOR output stays plain. - The MLflow column shows the MLflow run name (with a …<id8> fallback) instead of a truncated URL, reusing fetchMLflowRunName/mlflowRunLabel. Cached for terminal runs. - `i` opens a scrollable run-details pane and `L`/`l` a logs snapshot pane (bubbles/viewport); `esc` returns to the list. `enter` still opens MLflow. The logs snapshot is a one-shot tail (staticView) so an active run can't hang the pane. Co-authored-by: Isaac
Contributor
Approval status: pending
|
Collaborator
Integration test reportCommit: 1177284
8 interesting tests: 4 RECOVERED, 4 SKIP
Top 1 slowest tests (at least 2 minutes):
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Enriches the interactive
air listpicker toward the Python AIR CLI.Changes
NO_COLORoutput stays plain (no escapes).…<id8>fallback) instead of a truncated…/runs/<id>URL. Reuses the existingfetchMLflowRunName/mlflowRunLabel; the resolved label is cached for terminal runs.iandL/lopen an in-TUI scrollable pane —ishows run details (the same styled view asair get),Lshows a logs snapshot (bubbles/viewport).escreturns to the list with the cursor preserved.enterstill opens MLflow in the browser.Notes
staticView), so viewing an active run’s logs can’t hang the pane waiting on a live stream.renderRunText/fetchLogs) output; since the capture target isn’t a TTY they render in the ASCII profile (text and box borders intact, no color). A forced-color pane is a possible follow-up.mlflowExperimentURLfor the experiment link (no?o=, consistent with the other ML URLs) rather than adding a parallel helper.Tests
i→ detail-mode transition, detail-pane content +escback, run-name label.air list): the MLflow column now shows the run name; golden regenerated. Full air unit + acceptance suite,go vet,gofmt, andgolangci-lintall pass.This pull request and its description were written by Isaac.