Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor SMRT Cell pooling calculator to comply with the changes in SMRT Link 25.1 #582

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Karl-Svard
Copy link
Collaborator

Added

  • New flag --sequencing-container-udf
  • New flag --control-volume-udf

Changed

  • Updated the calculations to be based on the amount of SMRT cells to be loaded
  • The amount of control added to the pool is now calculated and set

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

@Karl-Svard
Copy link
Collaborator Author

Tested on stage

New config for automation Calculate Revio Pooling Volumes:

bash -c -l "conda activate cg_lims &&  lims -c /home/glsai/.genologics.yaml epps -l {compoundOutputFileLuid0} -p {processLuid} udf calculate revio-pooling --concentration-udf 'Concentration (ng/ul)' -v 'Volume of sample (ul)' -b 'Volume Buffer (ul)' --total-volume-udf 'Total Volume (ul)' --size-udf 'Size (bp)' --target-volume-udf 'Total Volume per SMRT Cell (ul)' --target-concentration-udf 'Target Loading Concentration (pM)' --sequencing-container-udf 'SMRT Cells to Load' --control-volume-udf 'Volume Sequencing Control (ul)'"

Success message:

Screenshot 2025-01-23 at 10 56 32

Fail message:

Screenshot 2025-01-23 at 11 04 23

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