knowledgesystems / cmo-pipelines

GNU Affero General Public License v3.0
3 stars 10 forks source link

Do not assume consent status has changed if germline fetch fails #1134

Closed jamesqo closed 6 months ago

jamesqo commented 6 months ago

Recently, the germline server at draco.mskcc.org has been returning an empty response for the consent values. This causes us to assume that all germline samples have been unconsented and remove them from the mskimpact mutations file, leading to https://github.com/knowledgesystems/pipelines-scrum/issues/1246.

This PR fixes that behavior by exiting early if no consent values are found.

/cc @averyniceday @callachennault