-
Notifications
You must be signed in to change notification settings - Fork 327
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
Show table groups without table rows #2103
Comments
@Nayananga this is a default by FluentUI's DetailsList the Wave is using but it can be changed on our side. @mturoci, it can be easily managed by specifying |
@Nayananga what's the point of showing empty groups? |
As I've mentioned above, these table data are fetching from a backend service, as how the ML works that may have models without registered model versions. |
That does not answer my question. Let me rephrase it: What's the UX value of showing empty groups? |
let me ping the UX team to answer it, @shihan007 |
Let me discuss this with @shereezhang and get back to you |
Hey thanks so much for considering this ticket @marek-mihok and @mturoci! In the MLOps App 'Models' tab, there's a table which lists all the Registered Model Versions in a Project, and currently we group them by the Parent Model (ie. first row is Version 1, second is Version 2, etc) to show model lineage across versions. (can see example in Nayananga's screenshot) In MLOps, it's possible to delete Model Versions without deleting the Parent Model (so the user can delete all rows but the table group should still exist cause the 'Parent' exists). We'd like to still be able to display this 'Parent Model' as an empty table group so the user knows that parent model still exists. Knowing the Parent Model exists in the UI is important because 1) the user may want to register new Model Versions (ie. rows) under it, or 2) we raise an error if a user tries to create Parent Models with duplicate names so the user should know what current Parent Model names are already used. Hopefully this is helpful! let me know if there are further questions. |
Thanks for the clarification @shereezhang, that makes sense.
@marek-mihok let's make it a new default. |
Wave SDK Version, OS
(e.g. 4.2.0, Windows)
0.26.1, MacOS
Actual behavior
(A clear and concise description of what happened.)
I want to show table groups that may or may not have table rows, But when I provide
rows=[]
to a particulartable_group
it doesn’t visible in the UI.But when I provide a
table_group
with atable_row
where it has an empty list forcells
it showsInvalid Date
text for corresponding empty values where thetable_column
hasdata_type="time"
Expected behavior
(A clear and concise description of what you expected to happen.)
Table groups should be able to exist even without table rows
Steps To Reproduce
sample code
The text was updated successfully, but these errors were encountered: