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.

Sign up for GitHub

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

Jump to bottom

Story: Limit number of metrics/params/deps/outs displayed wrt experiments #1536

Closed
mattseddon opened this issue Apr 7, 2022 · 2 comments
Closed

Story: Limit number of metrics/params/deps/outs displayed wrt experiments #1536

mattseddon opened this issue Apr 7, 2022 · 2 comments
Labels
A: experiments Area: experiments table webview and everything related enhancement New feature or request story Product feature aka epic. Discussion, progress, checkboxes for implementation, etc

Comments

@mattseddon
Copy link
Contributor

From testing with a DVCGen repository of 1000 metrics+params combined:

Screen.Recording.2022-04-05.at.9.29.27.am.mov

Some ideas to improve performance:

  1. Only display the first 100 metrics, first 100 params, and first 100 data fields in the table, leave the rest as unselected.
  2. Introduce a config option to only display the first 100 metrics, first 100 params, and first 100 data fields across the entire extension. Users would have to select their desired combination on the first load of a workspace for each repository within that workspace.

The first step is to introduce a telemetry event to be able to quickly confirm the size of our user base's repositories. We need to capture the number of metrics/params/deps/outs that could possibly be displayed. This will then let us know how much effort we need to spend trying to fix this issue.

Related to https://github.com/iterative/studio/issues/3257
@mattseddon mattseddon added enhancement New feature or request A: experiments Area: experiments table webview and everything related story Product feature aka epic. Discussion, progress, checkboxes for implementation, etc labels Apr 7, 2022
@mattseddon
Copy link
Contributor Author

Depends on #1183

@mattseddon mattseddon mentioned this issue May 24, 2022
Closed
@mattseddon mattseddon assigned mattseddon and unassigned mattseddon Jun 3, 2022
@mattseddon
Copy link
Contributor Author

Closing placeholder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: experiments Area: experiments table webview and everything related enhancement New feature or request story Product feature aka epic. Discussion, progress, checkboxes for implementation, etc
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mattseddon