Model Registry: Display Experiment in model card sidebar
What does this MR do and why?
This MR adds a field to the sidebar of the Model card in the Model Registry, the field will display the default experiment of the model with a link to that experiment in Model Experiments.
References
Please include cross links to any resources that are relevant to this MR. This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
![]() |
![]() |
How to set up and validate locally
- Open GDK: http://127.0.0.1:3000/flightjs/Flight/-/ml/models
- Create a model if one doesn't exist
- Open the model and observe in the sidebar the
Experiment
element with a link to the experiment.
Numbered steps to set up and validate the change are strongly suggested.
Related to #505473 (closed)
Merge request reports
Activity
changed milestone to %17.7
assigned to @fdegier
added 5 commits
-
7836730c...51563a71 - 4 commits from branch
master
- 802826b2 - Model registry: Add Experiment to model card
-
7836730c...51563a71 - 4 commits from branch
- A deleted user
added frontend label
2 Warnings featureaddition and featureenhancement merge requests normally have a documentation change. Consider adding a documentation update or confirming the documentation plan with the Technical Writer counterpart.
For more information, see:
- The Handbook page on merge request types.
- The definition of done documentation.
This merge request contains lines with testid selectors. Please ensure e2e:test-on-omnibus
job is run.testid
selectorsThe following changed lines in this MR contain
testid
selectors:app/assets/javascripts/ml/model_registry/apps/show_ml_model.vue
+ <div class="gl-text-lg gl-font-bold" data-testid="sidebar-experiment-title"> + <div class="gl-pt-2 gl-text-subtle" data-testid="sidebar-experiment-label"> + data-testid="sidebar-latest-experiment-link"
If the
e2e:test-on-omnibus
job in theqa
stage has run automatically, please ensure the tests are passing. If the job has not run, please start themanual:e2e-test-pipeline-generate
job in theprepare
stage and ensure the tests infollow-up:e2e:test-on-omnibus-ee
pipeline are passing.For the list of known failures please refer to the latest pipeline triage issue.
If your changes are under a feature flag, please check our Testing with feature flags documentation for instructions.
Reviewer roulette
Category Reviewer Maintainer frontend @jrushford
(UTC+1, same timezone as author)
@aturinske
(UTC-8, 9 hours behind author)
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by Ghost UserBundle size analysis [beta]
This compares changes in bundle size for entry points between the commits 700b9eca and cf4046fa
Special assetsEntrypoint / Name Size before Size after Diff Diff in percent average 4.41 MB 4.41 MB - 0.0 % mainChunk 3.28 MB 3.28 MB - 0.0 %
Note: We do not have exact data for 700b9eca. So we have used data from: 63f92bea.
The intended commit has no webpack pipeline, so we chose the last commit with one before it.Please look at the full report for more details
Read more about how this report works.
Generated by
DangerEdited by Ghost Useradded 224 commits
-
802826b2...83573a66 - 222 commits from branch
master
- 753787be - Model registry: Add Experiment to model card
- cc744dc9 - Display branch rule squash settings
-
802826b2...83573a66 - 222 commits from branch
added 5 commits
-
cc744dc9...9a766e0f - 4 commits from branch
master
- 51e1dbf2 - Model registry: Add Experiment to model card
-
cc744dc9...9a766e0f - 4 commits from branch
- Resolved by Eduardo Bonet
@eduardobonet can you do the frontend review please?
requested review from @eduardobonet
- Resolved by Fred de Gier
- Resolved by Fred de Gier
- Resolved by Andrew Fontaine
added 1 commit
- 2326d43b - Model registry: Add Experiment to model card
- Resolved by Fred de Gier
Thanks @eduardobonet addressed your comments, can you have another look?
requested review from @eduardobonet