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-23976: Use the base class implementations for gen3 defects and camera #263
Conversation
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.
One small comment comment, and a tie-back to obs_base.
filterDefinitions = HSC_FILTER_DEFINITIONS | ||
|
||
def __init__(self, **kwargs): | ||
super().__init__(**kwargs) | ||
packageDir = getPackageDir("obs_subaru") | ||
self.configPaths = [os.path.join(packageDir, "config"), | ||
os.path.join(packageDir, "config", "hsc")] | ||
os.path.join(packageDir, "config", self.policyName)] |
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.
Per my comment in the obs_base PR, self.policyName
here could probably be replaced with self.name.lower()
.
python/lsst/obs/subaru/instrument.py
Outdated
for defect, dataId in datasetRecords: | ||
butler.put(defect, datasetType, dataId) | ||
# Write defects | ||
self.writeStandardTextCuratedCalibrations(butler) |
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 comment is misleading, I think: it's writing more than just defects, it's whatever is in standardCuratedDatasetTypes
.
python/lsst/obs/subaru/instrument.py
Outdated
universe=butler.registry.dimensions) | ||
butler.registry.registerDatasetType(datasetType) | ||
# Write cameraGeom.Camera | ||
self.writeCameraGeom(butler) |
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.
I think the comment here is unnecessary, given the method name.
This will write defects and camera geom for us.
No description provided.