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-13353: Add FITS compression support to gen3 formatter #257
Conversation
Requires lsst/daf_butler#305 |
return recipe | ||
|
||
# @staticmethod | ||
def validateWriteRecipes(self, recipes): |
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.
@jonathansick points out that pydantic would be perfect here instead of hand crafted validation.
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.
Nicely done!
* quantizePad: number of stdev to allow on the low side (for | ||
STDEV_POSITIVE/NEGATIVE) | ||
* bscale: manually specified BSCALE (for MANUAL scaling) | ||
* bzero: manually specified BSCALE (for MANUAL scaling) |
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 four above are float
.
Should the types be in backticks?
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.
Oh. Hadn't thought about it since I copied it directly and never read it. Will see what the docs look like.
Also includes tests of lossless, no compression, and lossy. Uses the gen2 cameraMapper code and writeRecipes.yaml content.
No description provided.