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(weave): Runs History Tables Scrubber #3459

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nicholaspun-wandb
Copy link
Contributor

@nicholaspun-wandb nicholaspun-wandb commented Jan 22, 2025

Description

  • Addresses WB-22173
  • Introduces new panel to display a paginated table of the history tables for a given table name (this part isn't fully implemented yet)
  • The pagination is controlled by a slider and respects the _step value associated to each table version

Note: This is largely ready for initial review, but there's still some polishing needed:

  • As mentioned above, table selection isn't ready yet and will be provided in the config component (similar to run tables)
  • There's a bit of copied code between this panel and the run tables panel
  • That slider will eventually be centered 😅

Testing

Local FE

Happy Case

Screen.Recording.2025-01-22.at.8.55.00.AM.mov

Multiple tables

Coming soon ...

Irregular Steps

Coming soon ...

@circle-job-mirror
Copy link

circle-job-mirror bot commented Jan 22, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant