Skip to content

[WIP] Update saving options and remove fst references#311

Closed
Copilot wants to merge 1 commit intomasterfrom
copilot/remove-fst-and-update-saving-options
Closed

[WIP] Update saving options and remove fst references#311
Copilot wants to merge 1 commit intomasterfrom
copilot/remove-fst-and-update-saving-options

Conversation

Copy link

Copilot AI commented Mar 16, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

I want to change a few things about it. First of all, I would like you to save all quarters files to a single parquet file (that would be a list of the quarters dataframes) when downloaded, lazy-load the needed columnes to create the panels and the variables necessarie. Other thing I would like to change is that I want the options of the function to receive a vector with the following composition: save_options = (TRUE, TRUE) - the default, in which you save all files as csv. If (FALSE, TRUE) - saves quarters and panels as parquet. If (TRUE, FALSE) - saves only the panels in csv. else If (FALSE, FALSE) - saves only the panels as parquet. When saving in parquet, it should be a list of the panels. fst will no longer be used. I want you to remove all mentions of the fst, be it on the documentation or in the code. I recommend you to not test the function as it may take a long time.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

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.

2 participants