MazinLab / MKIDPipeline

The MKID Data Pipeline
http://web.physics.ucsb.edu/~bmazin/
6 stars 3 forks source link

Cast both sides of flag filter to uint64 #128

Closed ld-cd closed 5 days ago

ld-cd commented 5 days ago

This is not strictly safe if we have 64 flags,

A correct fix would be to identify why the self._flagArray Field is an int64 and not a uint64, and why sometimes the flags we want to filter int64 flags and address this issue

Closes #127

codecov[bot] commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 19.81%. Comparing base (ed1a59e) to head (bb84ad4). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #128 +/- ## ======================================= Coverage 19.81% 19.81% ======================================= Files 26 26 Lines 7579 7579 ======================================= Hits 1502 1502 Misses 6077 6077 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features: