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
Add DCR model data types #42
Conversation
a0f3ca0
to
6ce27ba
Compare
6b18f76
to
33bcc87
Compare
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.
Couple questions about the bypass methods. I'm not sure what to check in the .paf file.
return self._computeCoaddExposureId(dataId, True) | ||
|
||
def bypass_dcrCoaddId_bits(self, datasetType, pythonType, location, dataId): | ||
return 1 + 7 + 13*2 + 3 |
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.
Since this is copied from bypass_deepCoaddId_bits
, should it just call that instead?
@@ -236,6 +236,13 @@ def bypass_deepMergedCoaddId_bits(self, *args, **kwargs): | |||
def bypass_deepMergedCoaddId(self, datasetType, pythonType, location, dataId): | |||
return self._computeCoaddExposureId(dataId, False) | |||
|
|||
def bypass_dcrMergedCoaddId_bits(self, *args, **kwargs): | |||
"""The number of bits used up for patch ID bits""" | |||
return 64 - self._nbit_id |
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.
Should you just call bypass_deepMergedCoaddId_bits
directly hear instead?
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.
Yes, that would be much better. I'll correct all the bypass functions to call the deepCoadd version.
No description provided.