Skip to content

Fix console blank filtering threshold parsing - #773

Open
Fraximov wants to merge 1 commit into
systemsomicslab:masterfrom
Fraximov:agent/fix-console-blank-threshold
Open

Fix console blank filtering threshold parsing#773
Fraximov wants to merge 1 commit into
systemsomicslab:masterfrom
Fraximov:agent/fix-console-blank-threshold

Conversation

@Fraximov

Copy link
Copy Markdown

Summary

  • preserve the legacy SampleMaxOverBlankAverage value
  • also apply the console value to FoldChangeForBlankFiltering, which the blank filter reads
  • add a regression test for custom sample-max/blank-average thresholds
  • include the existing console test project in the solution so CI runs the regression test

Root cause

The console parser stored Sample max / blank average only in the legacy property, while alignment filtering reads FoldChangeForBlankFiltering. Both default to 5, hiding the mismatch until a custom threshold is configured.

Validation

  • dotnet test tests/MSDIAL5/MsdialCoreTestAppTests/MsdialCoreTestAppTests.csproj --configuration "Debug vendor unsupported" (5 tests passed)

@Fraximov
Fraximov marked this pull request as ready for review August 19, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant