Skip to content

Add support for extracting charts#986

Merged
varunfb merged 2 commits into
mainfrom
sp-charts-patch
Aug 24, 2025
Merged

Add support for extracting charts#986
varunfb merged 2 commits into
mainfrom
sp-charts-patch

Conversation

@subramen

@subramen subramen commented Aug 23, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Extract charts as an artifact
  • Save charts to DB, CSV and excel
  • Refactor CLI to take artifacts arg as a comma-separated string
  • Add docstrings

@subramen
subramen requested a review from varunfb August 23, 2025 02:35
@meta-cla meta-cla Bot added the cla signed label Aug 23, 2025
varunfb
varunfb approved these changes Aug 24, 2025
View reviewed changes

@varunfb varunfb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ship It !!

@varunfb
varunfb merged commit 784e63f into main Aug 24, 2025
2 of 4 checks passed
@meta-llama meta-llama deleted a comment from sithreach Aug 25, 2025
@subramen
subramen deleted the sp-charts-patch branch August 25, 2025 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@varunfb varunfb varunfb approved these changes

Assignees

No one assigned

Labels

cla signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@subramen @varunfb