sanger-archive / lighthouse-ui

Front end for lighthouse API
2 stars 4 forks source link

GPL-559 Automatically batch lighthouse sentinel samples #10

Closed rl15 closed 4 years ago

rl15 commented 4 years ago

User story GPL-559 | As genomics POs (Emma, Rich C & James) we would like Lighthouse sentinel samples to be automatically batched for cherry picking

Who are the primary contacts for this story Emma G Rich C James M Danni W

*Acceptance criteria To be considered successful the solution must allow:

*1 There maybe a better way to feedback that a plate and so its samples have been excluded from the mega batch than a table underneath (EG big red cross next to item /greyed out?). However this is working design for now.

Dependencies This story has dependency on GPL-558: https://github.com/sanger/lighthouse-ui/issues/2 And: https://github.com/sanger/sequencescape/issues/2837

Additional context See sequence flow diagram: https://ssg-confluence.internal.sanger.ac.uk/x/jqsKBg

rl15 commented 4 years ago

Mockup (mailed to SSR Tuesday, 14 July 2020 at 11:20): 20200714_GPL-559_Batch_Sentinal_CP.pdf

JamesGlover commented 4 years ago

This is a bit of a diversion from the current process for generating batches. Its one of the reasons I had hoped to get cherrypicking re-written at some point.

Hidden requirements:

Infrastructure in place:

Other things to consider:

JamesGlover commented 4 years ago

I'd suggest adding a 'Batched cherrypick' submission template, and then trigger that via the API. This would have the advantage that it would also expose the functionality within Sequencescape in general. That way we'd need:

JamesGlover commented 4 years ago

OOh! Better idea! New endpoint: pick-list which just takes an array of receptacles, and study information, and does all this under the hood. That way if we get around to substantially changing the way cherrypicking works we can do so in a nicely isolate manner. From a suggestion to @stevieing

Add a new pick-list endpoint, which allows a ‘post’ containing a list of receptacles, and probably also allow a study/project as well. This will generate all the necessary requests and submissions behind the scenes. The response will contain a pick-list name and link. These will provide a link to the batch.

emrojo commented 4 years ago

I think this feature (create a batch automatically) is mainly a change in SS so is going to be implemented in https://github.com/sanger/sequencescape/issues/2837 , I dont see what should be changed in Lighthouse/Crawler. I'll add the 'Question' flag for this reason.

JamesGlover commented 4 years ago

I think this feature (create a batch automatically) is mainly a change in SS so is going to be implemented in sanger/sequencescape#2837 , I dont see what should be changed in Lighthouse/Crawler. I'll add the 'Question' flag for this reason.

There is the front end component in the app @stevieing is delivering as part of GPL-558 (sanger/lighthouse-ui#2) which will mostly be the API call to Sequencescape, and displaying the returned link.

stevieing commented 4 years ago

Tasks

stevieing commented 4 years ago

Sequencescape endpoint:

{
          'data' => {
            'type' => 'pick_lists',
            'attributes' => {
              'asynchronous' => true,
              'labware_pick_attributes' => [
                { source_labware_id: plate_1.id },
                { source_labware_barcode: plate_2.machine_barcode },
              ]
            }
          }
        }