Add Pipeline Tooltips Read From Pipeline Headers - #1570
BryonLewis merged 3 commits into
Conversation
Sorry, something went wrong.
|
Within the latest published or even the master branch of the VIAME Repo there are no pipelines/configs that contain Description: Need some samples and pipelines before I can accurately test. |
Sorry, something went wrong.
Main already extracts the metadata; this just wires it through to the dropdown UIs that weren't surfacing it. - Solid dark background on .pipeline-description-tooltip in RunPipelineMenu so the text is readable. - MultiPipeline (desktop): tooltip on each pipeline showing metadata.description. - MultiTrainingMenu (desktop) + web-girder RunTrainingMenu: same treatment for TrainingConfig.description; switch to item-text/ item-value="name" and render item.name in the selection slot.
|
is this superseded by #1624 from @Louis-Pagnier-KW? If so this PR should probably be closed? |
Sorry, something went wrong.
|
I see now that this is slightly different from the previous pipeline description. Updated it to fix some minor issues with it running. Added a small 250ms delay for pipeline/trianing tooltips so because it seemed a bit hectic before. |
Sorry, something went wrong.
BryonLewis
left a comment
There was a problem hiding this comment.
Did some minor modifications and testing of the functionality. Seems to work properly.
Sorry, something went wrong.
Summary
Surface the pipeline / training-config description (read from each `.pipe` / `.conf` header) as a hover tooltip in the dropdown menus that don't already show it.
The header-extraction pipeline (`extract_pipe_metadata` server-side, `extractPipeMetadata` client-side) is already on main; this PR is just the UI wire-up that was missing.
Changes vs main
This rebase drops the original branch's `extract_pipe_description` / `PipelineDescription.description` plumbing because main now provides the same data through the richer `PipeMetadata` / `TrainingConfig.description` types.