Skip to content

Uncorrected ratio plots#257

Merged
RasmusBurge-CG merged 3 commits intointegration-branch-novaseqxfrom
uncorrected-ratio-plots
Apr 13, 2026
Merged

Uncorrected ratio plots#257
RasmusBurge-CG merged 3 commits intointegration-branch-novaseqxfrom
uncorrected-ratio-plots

Conversation

@islean
Copy link
Copy Markdown
Member

@islean islean commented Apr 8, 2026

Added

  • Parsing of uncorrected ratios
  • Box plots for uncorrected ratios

Changed

Fixed

Steps to consider while deploying

  • Configuration changes:
  • Documentation updates:
  • Inform users by email:

Review:

  • Code approved by
  • Tests executed on stage by: (Document the test done with screen shots and description.)
  • "Merge and deploy" approved by

This version is a:

  • MAJOR - when you make incompatible API changes
  • MINOR - when you add functionality in a backwards compatible manner
  • PATCH - when you make backwards compatible bug fixes or documentation/instructions

@islean islean changed the base branch from master to integration-branch-novaseqx April 8, 2026 13:21
@clingen-sthlm clingen-sthlm temporarily deployed to stage April 8, 2026 13:38 Inactive
@islean
Copy link
Copy Markdown
Member Author

islean commented Apr 8, 2026

Problem with current approach: The name of the batch is sent as a separate parameter, so in the beginning when we only have a single batch for example with uncorrected ratios, these are marked incorrectly (as belonging to another batch) in the frontend.

@clingen-sthlm clingen-sthlm temporarily deployed to stage April 9, 2026 07:19 Inactive
@clingen-sthlm clingen-sthlm temporarily deployed to stage April 9, 2026 08:39 Inactive
@islean islean marked this pull request as ready for review April 10, 2026 11:43
@islean islean requested a review from a team as a code owner April 10, 2026 11:43
Copy link
Copy Markdown

@RasmusBurge-CG RasmusBurge-CG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@RasmusBurge-CG RasmusBurge-CG merged commit fda7bb0 into integration-branch-novaseqx Apr 13, 2026
3 checks passed
@RasmusBurge-CG RasmusBurge-CG deleted the uncorrected-ratio-plots branch April 13, 2026 09: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.

3 participants