ON - Reason for sequencing updates

[GH-Emily]

Hi @sifavahora,

Could you please update the attached samples? Additional information was provided by PHO.

NML_PHO_reasons_for_sequencing_retrospective_2022-05-02_final_dpUpdate.csv

[edsu7]

Hi @GH-Emily,

The field purpose_of_sequencing in our schema is now a list.

Below is an example of the requested changed:	studyId	analysisId	submitterDonorId	old purpose_of_sequencing	new purpose_of_sequencing
PHO-ON	ec72df13-984f-4ad5-b2df-13984ffad587	ON_80757	International travel surveillance'	Targeted surveillance (non-random sampling)'

I would like to clarify that you want to replace the older value and not append the new value to the the "list".

Cheers, Edmund

[GH-Emily]

Hi @edsu7

I have attached a new update file. It now lists information in the purpose_of_Sequencing field as a list rather than splitting the two values into purpose_of_Sequencing and purpose_of_sequencing_details. Everything in the new file should overwrite the current values, rather than being concatenated with the existing value.

If there is an existing value in the field purpose_of_sequencing_details and my file has a null value, please over write the existing value with the null.

NML_PHO_reasons_for_sequencing_retrospective_2022-05-02_final_dpUpdate.csv

Thank you and sorry for the massive list!

Emily

[edsu7]

57569/57570 samples have had their purpose_of_sequencing and purpose_of_sequencing_details updated.

The odd sample ON_388855 was previously suppressed https://github.com/cancogen-virus-seq/portal/issues/321

[ghost]

@GH-Emily please close this if applicable.

[GH-Emily]

thank you