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

Create find_pointings.py #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

GrahamDoskoch
Copy link
Member

Added current version of find_pointings.py to repository. This implicitly performs some checks for .fits files (e.g. making sure that they're readable, properly encoded, and not empty.)

Added current version of find_pointings.py to repository. This implicitly performs some checks for .fits files (e.g. making sure that they're readable, properly encoded, and not empty.)
Found and fixed bug: since the values of the path and survey name aren't set until after parse_fits_or_fil() is called, they would always be erroneously categorized as missing.
I changed the way the script gets file size, and now have it actually writing the quantity to the pointing list, so it can be uploaded to the database.
Fixed one last bug.
To be tied together in a list, the file size needs to be a string, not a float; previously, it was stored as a float, which borked everything.
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.

None yet

1 participant