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

Metadata Modal is empty #3592

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

MVarshini
Copy link
Contributor

PBENCH-1313

Display metadata modal is empty

Added mine filter

Display metadata modal is empty
@MVarshini MVarshini added the Dashboard Of and relating to the Dashboard GUI label Jan 4, 2024
@MVarshini MVarshini requested review from webbnh and dbutenhof January 4, 2024 12:31
@MVarshini MVarshini self-assigned this Jan 4, 2024
Copy link
Member

@dbutenhof dbutenhof left a comment

Choose a reason for hiding this comment

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

This ought to help, at least in the short term, since the Overview page should always show a small subset of the total datasets.

As I mentioned yesterday, my implementation of keysummary obviously doesn't scale well, and I'll have to think about whether it's possible to improve on that. Or maybe it was just a bad idea in the first place and (at scale) it won't prove practical to present a full tree of all metadata keys in the global and user namespaces. 😦

Copy link
Member

@webbnh webbnh left a comment

Choose a reason for hiding this comment

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

👍

@dbutenhof dbutenhof merged commit 314ded3 into distributed-system-analysis:main Jan 4, 2024
4 checks passed
@webbnh webbnh mentioned this pull request Jan 18, 2024
webbnh pushed a commit that referenced this pull request Jan 19, 2024
Display metadata modal is empty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dashboard Of and relating to the Dashboard GUI
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants