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
{{ message }}
Repository navigation
[Bug] CARDINALITY on an MS1 condition raises KeyError: 'mzenumeration' #264
The MS2 path sets it before appending each per-value frame:
ms2_filtered_df["mzenumeration"] =i
The MS1 path (ms1_condition) has no equivalent assignment, so the column never exists by
the time the merge reads it.
Suggested fix
Set ms1_filtered_df["mzenumeration"] = i in the MS1 loop, mirroring the MS2 path.
The same gap is in ms2prec_condition
ms2prec_condition also appends a per-value frame without setting mzenumeration, so a
multi-valued MS2PREC carrying CARDINALITY should reach the same KeyError. Grepping the
three condition functions for the assignment finds it only in the MS2 product path. I read
that one rather than running it, so it is worth confirming before fixing.
Activity