-
Notifications
You must be signed in to change notification settings - Fork 2
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
Rewrite CEMS archiver to use quarterly partitions rather than state-year. #181
Merged
Merged
Changes from 11 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
7b805be
WIP needs testing, rewrite to use quarterly rather than state-year pa…
e-belfer 38098b8
Add handling for giant zipfiles in download_and_zip
e-belfer 8bd7fc7
Try a new way to write bytes to zipfile
e-belfer 7d5a4e1
Change file count # for legibility
e-belfer 0ddabae
Zip quarters by year
e-belfer 660dbe6
Fix merge conflicts
e-belfer 3414cf9
Revert "Zip quarters by year"
e-belfer d64a2e2
Merge branch 'main' into cems_quarterly
e-belfer 9669bc4
Reduce concurrency to prevent memory overload
e-belfer a9e3415
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 78bec3e
Merge branch 'main' into cems_quarterly
e-belfer 64428e4
Standardize to year-quarter
e-belfer ff1fcb0
Correct year-quarter for missed logging statement
e-belfer e7d2ffc
update sandbox concept doi
e-belfer 8cd0170
Merge branch 'main' into cems_quarterly
e-belfer 45280b4
Merge branch 'main' into cems_quarterly
e-belfer fd3d3a0
Merge branch 'main' into cems_quarterly
e-belfer 3f52550
Merge branch 'main' into cems_quarterly
e-belfer File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did it end up just being impossible to run this archive via GitHub Actions? What's the bottleneck? Would we be able to run it on a 4CPU 16GB runner?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't tried yet, let me see! I think the Github action we'd want to use is getting defined in #197, and the existing action will fail because of the sandbox DOIs having changed.
Relatedly, @jdangerx our branches are in a bit of an intertwined state so let me know how you want to handle merge order in this instance?