Add JSON export button for line plots and upgrade gradio dependency#378
Merged
Conversation
- Added export functionality to download line plot data as JSON - Each plot now has an 'Export as JSON' button below it - JSON format includes x-axis name, y-axis name, and data for each run - Fixes enhancement request to allow exporting line plots as JSON data
Member
|
Thanks @JamshedAli18 for the PR. While this approach works, there's a cleaner UI using https://www.gradio.app/main/guides/custom-buttons. The Gradio team is still adding custom button support for gr.LinePlot, once this has been done, I'll reopen this PR and we can make the change. |
Contributor
🪼 branch checks and previews
|
Contributor
🦄 change detectedThis Pull Request includes changes to the following packages.
✅ Changeset approved by @abidlabs
|
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
Member
|
Going to merge this in, thanks @JamshedAli18! |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thank you for your contribution! All PRs should include the following sections. PRs missing these sections may be closed immediately.
Short description
This PR This PR adds the ability to export line plot data as JSON. Each plot now has an "Export as JSON" button that downloads the plot data in a structured format containing the x-axis name, y-axis name, and values for each run.
AI Disclosure
We encourage the use of AI tooling in creating PRs, but the any non-trivial use of AI needs be disclosed. E.g. if you used Claude to write a first draft, you should mention that. Trivial tab-completion doesn't need to be disclosed. You should self-review all PRs, especially if they were generated with AI.
Type of Change
Related Issues
If this PR closes an issue, please link it below:
Closes: #368
Closes: #385
Testing and linting
Please run tests before submitting changes:
and format your code using Ruff:
ruff check --fix --select I && ruff format