-
Notifications
You must be signed in to change notification settings - Fork 6
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
DM-18051: Get defect machinery working for the AuxTel #91
Merged
Commits on Apr 3, 2019
-
Configuration menu - View commit details
-
Copy full SHA for 3a50da5 - Browse repository at this point
Copy the full SHA 3a50da5View commit details -
Configuration menu - View commit details
-
Copy full SHA for c648f43 - Browse repository at this point
Copy the full SHA c648f43View commit details -
Configuration menu - View commit details
-
Copy full SHA for 37aad36 - Browse repository at this point
Copy the full SHA 37aad36View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7fea146 - Browse repository at this point
Copy the full SHA 7fea146View commit details -
Configuration menu - View commit details
-
Copy full SHA for b86e505 - Browse repository at this point
Copy the full SHA b86e505View commit details -
Configuration menu - View commit details
-
Copy full SHA for c1d351e - Browse repository at this point
Copy the full SHA c1d351eView commit details -
Add SConstruct instruction to run auxTel/SConscript
This builds defects for auxTel
Configuration menu - View commit details
-
Copy full SHA for ffa1fbe - Browse repository at this point
Copy the full SHA ffa1fbeView commit details -
Ignore the output .fits defect files Ignore the defectRegistry
Configuration menu - View commit details
-
Copy full SHA for f487f44 - Browse repository at this point
Copy the full SHA f487f44View commit details -
Add code to generate defect files and registry
Add a SConscript file which calls genDefectFits.py and genDefectRegistry.py to build fits files from the defect.dat files, and create a defect registry at scons-time. This is following the pattern (and copying the code) in obs_subaru. It is deliberately not moving this code into obs_base nor implementing this for other parts of obs_lsst as this is a pattern we want to change, but there is no point in doing this before Gen3 or fixing up calibration stuff in general.
Configuration menu - View commit details
-
Copy full SHA for cb0047f - Browse repository at this point
Copy the full SHA cb0047fView commit details -
Change to using detector as ccd key
Change the key used Change the keys stored in the registry to be careful Store both the name and number of the detector in the defect fits file
Configuration menu - View commit details
-
Copy full SHA for 85a0c98 - Browse repository at this point
Copy the full SHA 85a0c98View commit details -
Turn on defects now that we have them
Do this in auxTel.py so that it applies everywhere Also, fix a PEP8 violation in the spacing while there
Configuration menu - View commit details
-
Copy full SHA for 14b2ba1 - Browse repository at this point
Copy the full SHA 14b2ba1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 46b2492 - Browse repository at this point
Copy the full SHA 46b2492View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.