Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • SaQC SaQC
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 37
    • Issues 37
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • rdm-software
  • SaQCSaQC
  • Merge requests
  • !580

`flagByStatLowPass` is overwriting existing flags

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open David Schäfer requested to merge fix_noise into develop Nov 28, 2022
  • Overview 3
  • Commits 2
  • Pipelines 1
  • Changes 2

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.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_noise