Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add SaveToFile component for DataFrame, Data and Message exports #6114

Merged
merged 9 commits into from
Feb 17, 2025

Conversation

Cristhianzl
Copy link
Member

This pull request introduces a new component for saving various types of data to files in different formats. The SaveToFileComponent class has been added to handle DataFrames, Data, and Messages, providing options for saving them in formats such as CSV, Excel, JSON, and Markdown.

Key changes include:

New Component Addition:

Methods for Saving Data:

  • Implemented methods _save_dataframe, _save_data, and _save_message to handle the saving process for each data type, supporting multiple file formats.

image

…e DataFrames, Data, or Messages to various file formats. This component allows users to select the input type, file format, and file path for saving the data.
@Cristhianzl Cristhianzl self-assigned this Feb 4, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Feb 4, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 4, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 4, 2025
@ogabrielluiz
Copy link
Contributor

Hey @Cristhianzl

All Component PRs need tests. Could you, please, add them?

…nd consistency

🐛 (save_to_file.py): handle unsupported input types and formats by raising ValueErrors with informative error messages
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 12, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 12, 2025
Cristhianzl and others added 2 commits February 12, 2025 15:06
…Component to ensure proper saving of data to various file formats and handling of different input types.
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 12, 2025
…age text in the SaveToFileComponent class to ensure proper saving to file

🔧 (test_save_to_file_component.py): Refactor test cases in the SaveToFileComponent test file for better readability and maintainability
@github-actions github-actions bot removed the enhancement New feature or request label Feb 12, 2025
@github-actions github-actions bot added enhancement New feature or request and removed enhancement New feature or request labels Feb 12, 2025
@Cristhianzl
Copy link
Member Author

Hey @Cristhianzl

All Component PRs need tests. Could you, please, add them?

Done! Thanks!

Copy link
Contributor

@rodrigosnader rodrigosnader left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 14, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Feb 17, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 17, 2025
@Cristhianzl Cristhianzl added this pull request to the merge queue Feb 17, 2025
Merged via the queue into main with commit 8c74ead Feb 17, 2025
41 of 42 checks passed
@Cristhianzl Cristhianzl deleted the cz/save-to-file branch February 17, 2025 12:46
@Empreiteiro
Copy link

Thanks @Cristhianzl! This component was much needed!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants