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-18643: Rename referenceSelection to referenceSelector and change default field to "resolved" #98
Conversation
Your commit message first line is a bit too long. |
I guess I clean that up in a rebase? |
Interactive rebase and then |
config/imsim/singleFrameDriver.py
Outdated
config.processCcd.calibrate.astrometry.referenceSelection.unresolved.name = 'isresolved' | ||
config.processCcd.calibrate.astrometry.referenceSelection.unresolved.minimum = None | ||
config.processCcd.calibrate.astrometry.referenceSelection.unresolved.maximum = 0.5 | ||
config.processCcd.calibrate.astrometry.referenceSelector.doUnresolved = True |
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.
Having the configs duplicated between ProcessCcd and here could lead to different behaviors between the pipelines if the configs change. Take a look at https://github.com/lsst/obs_subaru/blob/master/config/singleFrameDriver.py for an example of how to handle this.
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.
Good point!
config/phosim/singleFrameDriver.py
Outdated
config.processCcd.calibrate.astrometry.referenceSelection.unresolved.name = 'isresolved' | ||
config.processCcd.calibrate.astrometry.referenceSelection.unresolved.minimum = None | ||
config.processCcd.calibrate.astrometry.referenceSelection.unresolved.maximum = 0.5 | ||
config.processCcd.calibrate.astrometry.referenceSelector.doUnresolved = True |
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.
Same here.
The field was called "isresolved" previously due to a miscommunication. This has been fixed at the same time as updating the config parameter name from referenceSelection to referenceSelector.
5610ccb
to
428c829
Compare
The field was called "isresolved" previously due to a miscommunication. This
has been fixed at the same time as updating the config parameter name.