-
Notifications
You must be signed in to change notification settings - Fork 33
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
Y24-382 - Research - updating volume after cherrypicking #4412
Y24-382 - Research - updating volume after cherrypicking #4412
Comments
Subject to verification, it appears that the volume reduction is triggered by https://github.com/sanger/sequencescape/blob/master/app/models/cherrypicking_pipeline.rb#L15 and clearly not in https://github.com/sanger/sequencescape/blob/master/app/models/pipeline.rb#L139. The actual calculation occurs in https://github.com/sanger/sequencescape/blob/master/app/models/cherrypick_request.rb#L19. |
Confirmed that the code in Once this has been completed, then a Robot verification completed and source volumes updated. event appears at the bottom of the Batch page. The destination plate is created and updates as soon as the Approve Plate Layout step has been completed. The confluence page Cherrypicking options in Sequencescape has been updated with this info. |
Calculations (and explanations) for volumes can be found in |
Given the table of results from test cherrypicks below, there appear to be some bugs:
Assuming simple arithmetic (which might not be the case), there are some unexpected final volumes. For plate SQPD-10857 with an initial volume of 50 ul, a 10 ul pick with no buffer, results in a plate of 40 ul and 10 ul - this is expected. For plate SQPD-10859 with an initial volume of 50 ul, a 10.71 ul pick with 5 ul buffer results in a plate of 39.29 ul (correct), but 15.0 ul in the destination plate - this is expected to be 15.71 ul. Similarly, for plate SQPD-10861 with an intial volume of 50 ul, a 14.29 ul pick with no buffer results in a plate of 35.71 ul (correct), but 10.0 ul in the destination plate - this is expected to be 14.29 ul. Plate SQPD-10864 replicates the issue identified in the training SS spreadsheet where the volume pick is Plates SQPD-10866 and SQPD-10868 attempt to replicate the inverse, to get 1 ul of buffer picked. In both cases 50 ul of well volume and no buffer is picked - which appears correct but would result in a greater quantity than expected - which is also correct. |
This is largely complete, only the questions below to Liz C are outstanding. The responses to these will tell us if there are any bugs in the calculations:
|
A bit of extra detail after reading the code that updates the source well volume:
@StephenHulme can you confirm if this is correct? |
Thanks for this investigation, it looks great. Looks like there is a bug in the setting of destination plate well volumes.
That plate is missing from your table, could you add it? Could you describe a bit more about the "issue identified in the training SS spreadsheet where the volume pick is less than 1 ul", please? One more question - was the one mentioned above the only issue raised by Liz in her spreadsheet, or are there more to look into? Thanks. |
Sorry, copy-paste error. Plate SQPD-10864 has been added to the table.
Correct, the volume pick (1 ul) is less than the specified 'robot minimum picking volume', in this instance of (5 ul). I think a typo crept in there...
As I understand the accompanying email there were two concerns associated with the spreadsheet. The first (I think, though could be wrong) was confirmation of volume and total micrograms changing, rather than a specific bug. The second concern was a potential bug with robot minimum picking volume which I have been able to replicate. While it appears clear that this behaviour is incorrect, there is currently fairly complex logic regarding the minimum picking volume making it challenging to identify the correct behaviour without additional information (see question 1 above). |
Thanks for the answers to my questions. Looks like we're just waiting on some clarification from Liz on your 2 q's then (v or v+b, and destination volume). |
Description
As an SSR (Liz C) I would like to know when well volumes in Sequencescape are updated after cherrypicks, so that I am confident when re-picking from a well that the remaining volume shown is correct.
Acceptance Criteria
The following outputs from the research are produced:
Who the primary contacts are for this work
Katy
Knowledge or Stake holders
Liz C
Additional context or information
Related stories:
The text was updated successfully, but these errors were encountered: