Skip to content
Snippets Groups Projects

`flagByStatLowPass` is overwriting existing flags

Merged David Schäfer requested to merge fix_noise into develop
All threads resolved!

The function flagByStatLowPass works by checking a certain condition window wise. If a certain conditions is met, the entire window is flagged, which might overwrite exiting flags. This MR fixes this unintended behavior.

Merge request reports

Pipeline #161529 passed

Pipeline passed for 44b468cd on fix_noise

Test coverage 77.00% (0.00%) from 1 job
Approval is optional

Merged by Bert PalmBert Palm 2 years ago (Apr 12, 2023 3:22pm UTC)

Merge details

  • Changes merged into with c824a270.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #163553 passed

Pipeline passed for c824a270 on develop

Test coverage 77.00% (0.00%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading