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-15928: Propagate DIASource flags into PPDB #35
Conversation
Edit formating of bit pack yaml file. Edit formating of bit pack to mimic ppdb schema formating.
Add unittest for bit-packing. Fix bugs in bit-packing tests. Change ValueError to KeyError
Debug integer bit unpacker.
Fix typo. Fix grammar in comment. Fix formating in afw doc.
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.
A couple of small things only.
@@ -24,14 +24,20 @@ | |||
""" | |||
|
|||
__all__ = ["MapApDataConfig", "MapApDataTask", | |||
"MapDiaSourceConfig", "MapDiaSourceTask"] | |||
"MapDiaSourceConfig", "MapDiaSourceTask", | |||
"UpackPpdbFlags"] |
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.
Can we call this UnpackPpdbFlags
instead? (here and throughout)
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.
Opps, that was a typo. Fixed.
def _create_bit_pack_mappings(self): | ||
"""Setup all flag bit packings. | ||
""" | ||
output_columns = [] |
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 you can omit the temporary variable output_columns and just use self.bit_pack_columns
directly
""" | ||
|
||
def __init__(self, flag_map_file, table_name): | ||
output_columns = [] |
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, I think you can omit the temporary variable output_columns
and just use self.bit_pack_columns
directly
Add more slot mappings. Fix typo in UnpackPpdbFlags name. Remove tempoarary output_columns variable.
1a064fa
to
692af66
Compare
No description provided.