-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Group colours by regex of the experiment name (UI). #6847
Conversation
Build is failing now. Have you confirmed that TAP presubmit has passed? Oh I think |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting code! Thanks for also cleaning up issues in the existing feature.
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_container.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_container.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_container.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_component.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog.ng.html
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/runs_group_menu_button_component.ng.html
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_container.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog.ng.html
Outdated
Show resolved
Hide resolved
Now the only part left is to make a commit for feature flag. I will try to make it in this PR as well. |
tensorboard/webapp/runs/views/runs_table/runs_group_menu_button_component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updates! Just a few more items
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_component.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_component.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_container.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/runs_group_menu_button_component.ts
Outdated
Show resolved
Hide resolved
927c819
to
e9d038a
Compare
tensorboard/webapp/runs/views/runs_table/runs_group_menu_button_container.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/runs_group_menu_button_component.ts
Outdated
Show resolved
Hide resolved
tensorboard/webapp/runs/views/runs_table/regex_edit_dialog_container.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New changes look great, I'll approve as soon as the CI issues are fixed (please request re-review when it's green)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! please merge this PR if CI passes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks for the keras workaround!
This was missing when #6847 was merged, but it's causing some build errors in internal repo due to more strict dependency checks (this target is currently used via transitive dependencies).
NOTE: This PR is a continuation of #6846.
Motivation for features / changes
Users want to color runs by the regex string of the corresponding to runs experiment names.
Technical description of changes
Screenshots of UI changes (or N/A)
N/A since internal change.
Detailed steps to verify changes work correctly (as executed by you)
Regex
.Experiment Name
in dropdown.Alternate designs / implementations considered (or N/A)
N/A