-
Notifications
You must be signed in to change notification settings - Fork 7
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
Task to add eBOSS-like redshift errors to catalog #149
Conversation
… set std of added errors
This code is taken from Seth's routines that were previously in a ch_pipeline pull request, but has been mildly refactored to put it into a separate task.
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.
A few small changes, otherwise very nice!
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.
This looks great, thanks for pulling it all together. I just have one comment.
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.
A few minor suggestions, but you can probably ignore them if you want.
Co-authored-by: Richard Shaw <richard@phas.ubc.ca>
Co-authored-by: Richard Shaw <richard@phas.ubc.ca>
Looks good to me! |
This refactors Seth's eBOSS redshift error code into a separate task, and implements a few other minor tweaks.