New script: Saphyr average concentration and CV calculation

[idalindegaard]

Added

New script to calculate average concentration and CV value for Saphyr samples

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

[idalindegaard]

Testing the script on stage with positive, negative and zero/no values:

All positive values:

One negative value:

One zero value:

One artifact with no values:

Three artifacts with no/zero/negative values plus the log file with more information (marked pink in picture) about which sample IDs (and which Concentration 1-3) were affected:

The tests show that the script counts all artifacts that have no/zero/negative values for any of the Concentrations 1-3 (ng/ul) and lets the user know it, you can read more in the log file about which samples had invalid values. In case this happens, the scripts stops after it has checked all the concentrations and won't continue to calculate the CV och Average concentration (ng/ul)

Conclusion: The script works as intended.

[Karl-Svard]

Very nice!! 🔥🔥🔥

I think it's ready for merging. Unless of course you have some other feature you'd like to add first

[idalindegaard]

Very nice!! 🔥🔥🔥

I think it's ready for merging. Unless of course you have some other feature you'd like to add first

Thanks @Karl-Svard! 😀

It says I have some conflicts that need to be resolved. Do you understand what to do?

[Karl-Svard]

Yeah, some of the recent PRs are modifying the same lines in the base.py of the calculate EPPs. But it can easily be fixed in the browser. We can go through it together when you have the time :)