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

Enable access to post-processed data #515

Merged
merged 18 commits into from
Mar 7, 2024
Merged

Conversation

ddobie
Copy link
Collaborator

@ddobie ddobie commented Jan 20, 2024

Fix #460.
Fix #509.

To do:

  • Add args to Query and find_sources.py to allow users to specify they would like to query the processed data
  • Add correct directory paths
  • Add correct file names (e.g. post-processed images will be in .fits.fz files.
  • Update defaults - the post-processed data should become the default.
  • Remove warning regarding "corrected" data not existing.
  • Update docs.
  • Update example notebooks

To do testing:

  • Test directory paths
  • Test correct file names
  • Test any new exceptions

@pep8speaks
Copy link

pep8speaks commented Jan 20, 2024

Hello @ddobie! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 525:80: E501 line too long (80 > 79 characters)

Comment last updated at 2024-03-07 05:07:36 UTC

@ddobie ddobie marked this pull request as ready for review March 7, 2024 05:14
@ddobie ddobie merged commit 7aa0d1a into dev Mar 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Enable access to post-processed data Make corrected data argument clearer for users
2 participants