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
@hrjoo and I discovered that we only saw duplicate events on channels there had been a reassignment of events (to that channel) with a large delta_t (typically -18 samples, larger than the detect_interval of 10). We suspect that such large time adjustments are incorrect. Probably there needs to be some dependency on detect_interval in or near this line:
NB: we still see duplicate events in ml_ms4alg v.0.2.1 (with the per-neighborhood duplicate event deletion provided by 0a8e350). This is probably because that fix does not address duplication across neighborhoods.
The text was updated successfully, but these errors were encountered:
@hrjoo and I discovered that we only saw duplicate events on channels there had been a reassignment of events (to that channel) with a large delta_t (typically -18 samples, larger than the detect_interval of 10). We suspect that such large time adjustments are incorrect. Probably there needs to be some dependency on detect_interval in or near this line:
ml_ms4alg/ml_ms4alg/ms4alg.py
Line 429 in 65f8624
NB: we still see duplicate events in ml_ms4alg v.0.2.1 (with the per-neighborhood duplicate event deletion provided by 0a8e350). This is probably because that fix does not address duplication across neighborhoods.
The text was updated successfully, but these errors were encountered: