icgc-argo / argo-meta

This repo is used to record metadata for testing workflows
0 stars 0 forks source link

OCCAMS-GB payload adjustments #44

Open lindaxiang opened 2 years ago

lindaxiang commented 2 years ago
edsu7 commented 2 years ago

Comment moved to more appropriate ticket https://github.com/icgc-argo/workflow-roadmap/issues/248

edsu7 commented 2 years ago

Comment moved to more appropriate ticket https://github.com/icgc-argo/workflow-roadmap/issues/248

edsu7 commented 2 years ago

Added ega related IDs to payloads: https://github.com/icgc-argo/argo-meta/blob/660511620297ff6f9885602733723f5854f4bb6b/icgc_song_payloads/OCCAMS-GB/WGS/trial_batch/EGAR00001130626_EGAF00000235882.json#L42-L61

b-f-chan commented 2 years ago

Feedback from Dev team is this field not used specifically, but want to reserve it for future use

From Bioinfo perspective, let's NOT use it for EGA data transfer, so we can still distinguish between ICGC25K carryover or not

b-f-chan commented 2 years ago

Sent sample registration table to Calvin for review, but one discrepancy found, awaiting feedback

If Calvin agrees to correction, we are good and can complete the submission