Skip to content
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

Updating preparation scripts and configurations #69

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Conversation

katia-tymofyeyeva
Copy link
Contributor

Updated the my_sites.txt file according to our new conventions, based on changes from Adrian. Implemented Adrian's changes to ARIA_prep, to only crop or stitch data using data from the products directory that falls within the defined time span.

…on changes from Adrian. Updated ARIA_prep to only crop or stitch data using data from the products directory that falls within the defined time span
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

prep/ARIA_prep.ipynb Show resolved Hide resolved
prep/ARIA_prep.ipynb Show resolved Hide resolved
prep/ARIA_prep.ipynb Show resolved Hide resolved
Copy link
Contributor Author

No changes needed for now... let's just get the functionality in the notebook, and maybe we can figure out how to streamline it better later (e.g., the best way to handle ariaDownload commands)

Copy link
Collaborator

@EJFielding EJFielding left a comment

Choose a reason for hiding this comment

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

This is OK, but now you merged the other changes to the Secular notebook, so you need to merge the changes in this PR with that merged PR.

@katia-tymofyeyeva katia-tymofyeyeva merged commit 32ee861 into main Feb 13, 2025
1 check passed
@katia-tymofyeyeva katia-tymofyeyeva deleted the update_prep branch February 13, 2025 02:18
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.

3 participants