-
Notifications
You must be signed in to change notification settings - Fork 4
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
Flux calibration #287
Flux calibration #287
Conversation
…nverting electrons to flux
I made several assumptions:
|
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.
Overall this is looking good to me, please see the few comments.
I've also reviewed the associated UT Tests in the DRP Test Definition page. Were the tests for "Tests to determine apparent magnitude in obs band" ever approved by anyone? I think the PR has been merged, right?
The answers are No, not yet approved, and yes, has been merged |
Ok, great, thanks for letting me know. @semaphoreP did you look this over before accepting the PR, or should I still take a look? |
I haven't looked at the UT Test Definitions for this test, so if you can review them, that'd be great. |
Describe your changes
Type of change
Reference any relevant issues (don't forget the #)
#7 , #241
Checklist before requesting a review