You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Toast observations include all detectors and samples, and use bitflags to handle cut detectors / samples. When we run the preprocessing pipeline inside the LoadContext operator, some detectors may be removed from the axismanager and a symmetric buffer of samples on either end of the data may also be removed.
The solution is to get the detector list from the det LabelAxis and use that to map to the full detector set in the toast observation. Then data can be copied one detector at a time. Currently entire 2D blocks of detectors/samples are copied at once, which results both in incorrect ordering of detector data and also a segfault when trying to access detector indices in the axismanager which are now out of range.
For the removed samples, we can assume (?!?) that the number of samples removed is symmetric and center the array of samples in the full array while flagging the missing samples on either end.
Toast observations include all detectors and samples, and use bitflags to handle cut detectors / samples. When we run the preprocessing pipeline inside the
LoadContext
operator, some detectors may be removed from the axismanager and a symmetric buffer of samples on either end of the data may also be removed.The solution is to get the detector list from the det LabelAxis and use that to map to the full detector set in the toast observation. Then data can be copied one detector at a time. Currently entire 2D blocks of detectors/samples are copied at once, which results both in incorrect ordering of detector data and also a segfault when trying to access detector indices in the axismanager which are now out of range.
For the removed samples, we can assume (?!?) that the number of samples removed is symmetric and center the array of samples in the full array while flagging the missing samples on either end.
This is the last thing holding up #943 .
The text was updated successfully, but these errors were encountered: