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-6640: IsrTask is not a valid CmdLineTask #152
Conversation
@@ -13,6 +14,9 @@ | |||
config.doDark = True # Required especially around CCD 33 | |||
config.doFringe = True | |||
config.fringe.filters = ['y', 'N921', 'N926', 'N973', 'N1010'] | |||
|
|||
# Do not use NO_DATA pixels for fringe subtraction. | |||
config.fringe.stats.badMaskPlanes = ['SAT', 'NO_DATA'] |
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 like an unrelated functional change that should be in a separate commit and justified.
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 was pulled from config/hsc/processCcd.py
, which listed these values for config.isr.fringe.stats.badMaskPlanes
. Rereading the version on master, I'm now not convinced that the += ["SUSPECT"]
line isn't the change in functionality. I will try to find a test.
config/hsc/isrWrap.py
Outdated
from lsst.utils import getPackageDir | ||
from lsst.obs.subaru.isr import SubaruIsrTask | ||
|
||
ObsConfigDir = os.path.join(getPackageDir("obs_subaru"), "config", "hsc") |
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.
Leading Uppercase is reserved for class names.
config.charImage.measurePsf.starSelector["objectSize"].widthMin = 0.9 | ||
config.charImage.measurePsf.starSelector["objectSize"].fluxMin = 4000 | ||
for refObjLoader in (config.calibrate.astromRefObjLoader, | ||
config.calibrate.photoRefObjLoader, | ||
config.charImage.refObjLoader, | ||
): | ||
refObjLoader.load(os.path.join(hscConfigDir, "filterMap.py")) | ||
refObjLoader.load(os.path.join(ObsConfigDir, "filterMap.py")) |
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.
The changes in this file seem unnecessary.
config.calibrate.measurement.load(os.path.join(configDir, "hsm.py")) | ||
config.calibrate.measurement.load(os.path.join(ObsConfigDir, "apertures.py")) | ||
config.calibrate.measurement.load(os.path.join(ObsConfigDir, "kron.py")) | ||
config.calibrate.measurement.load(os.path.join(ObsConfigDir, "hsm.py")) |
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.
Again, changes in this file are unnecessary.
config/suprimecam-mit/isrWrap.py
Outdated
from lsst.utils import getPackageDir | ||
from lsst.obs.subaru.isr import SuprimeCamIsrTask | ||
|
||
ObsConfigDir = os.path.join(getPackageDir("obs_subaru"), "config", "suprimecam") |
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.
"suprimecam-mit"
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.
Thanks for catching this.
5e4d42a
to
5bc46c5
Compare
5bc46c5
to
a4cc1af
Compare
The IsrTask was not usable as a CmdLineTask, as it was written to be called as a subtask of ProcessCcdTask. This allows the default IsrTask to be retargeted to camera-specific versions of the ISR processing. A new RunIsrTask now exists to hold IsrTask for retargetting in standalone processing, and both processCcd.py and runIsr.py now read from the same configuration files for each obs_ package. This has involved migrating configuration values from config/processCcd.py to config/isr.py for many obs_ packages.
a4cc1af
to
8a466cd
Compare
This ticket adds a stand-alone runIsr.py script, as well as ensuring that runIsr.py and processCcd.py read the same ISR configuration files.