Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: show metadata in run table #9776

Merged
merged 2 commits into from
Aug 2, 2024
Merged

Conversation

keita-determined
Copy link
Contributor

@keita-determined keita-determined commented Jul 31, 2024

Ticket

ET-222
ET-223

Description

Show metadata in the run table.
have not excluded the array type columns yet. will implement it after #9759 is merged, but its not a blocker to merge this PR.

Test Plan

  • Go to run table
  • Click Columns button and see Metadata tab. user should be able to add metadata columns and filter and sort by metadata (metadata with array type are not supported for filtering and sorting, PR will come after feat: add array column type for abitrary metadata #9759 to exclude the array type metadata in filter and sort)

Checklist

  • Changes have been manually QA'd
  • New features have been approved by the corresponding PM
  • User-facing API changes have the "User-facing API Change" label
  • Release notes have been added as a separate file under docs/release-notes/
    See Release Note for details.
  • Licenses have been included for new code which was copied and/or modified from any external code

@cla-bot cla-bot bot added the cla-signed label Jul 31, 2024
@keita-determined keita-determined marked this pull request as ready for review July 31, 2024 21:21
@keita-determined keita-determined requested a review from a team as a code owner July 31, 2024 21:21
Copy link

netlify bot commented Jul 31, 2024

Deploy Preview for determined-ui ready!

Name Link
🔨 Latest commit 89ad92e
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/66abe88fc8e2c20008807f78
😎 Deploy Preview https://deploy-preview-9776--determined-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@EmilyBonar EmilyBonar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2024-08-01 at 14-03-45 Uncategorized Runs - Determined
This is really bad UX, needs some design work to figure out correct solution.

webui/react/src/pages/FlatRuns/FlatRuns.tsx Outdated Show resolved Hide resolved
@keita-determined keita-determined force-pushed the feature/metadata-run-table branch from 91aa01b to 89ad92e Compare August 1, 2024 19:57
@keita-determined
Copy link
Contributor Author

@EmilyBonar
image

fixed it

@@ -18,3 +18,9 @@
white-space: nowrap;
}
}

@media only screen and (width <= 390px) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like $breakpoint- variables are not working so use the constant number

Copy link

codecov bot commented Aug 1, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 49.73%. Comparing base (61aad78) to head (89ad92e).
Report is 2 commits behind head on main.

Files Patch % Lines
webui/react/src/pages/FlatRuns/FlatRuns.tsx 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9776      +/-   ##
==========================================
- Coverage   54.04%   49.73%   -4.32%     
==========================================
  Files        1260      937     -323     
  Lines      155551   126228   -29323     
  Branches     3501     3502       +1     
==========================================
- Hits        84073    62779   -21294     
+ Misses      71332    63303    -8029     
  Partials      146      146              
Flag Coverage Δ
harness ?
web 53.23% <83.33%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...eact/src/components/ColumnPickerMenu.test.mock.tsx 100.00% <100.00%> (ø)
webui/react/src/components/ColumnPickerMenu.tsx 95.89% <100.00%> (+0.02%) ⬆️
webui/react/src/pages/FlatRuns/FlatRuns.tsx 11.08% <0.00%> (-0.02%) ⬇️

... and 323 files with indirect coverage changes

Copy link
Contributor

@EmilyBonar EmilyBonar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@keita-determined keita-determined merged commit fe14062 into main Aug 2, 2024
80 of 95 checks passed
@keita-determined keita-determined deleted the feature/metadata-run-table branch August 2, 2024 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants