Skip to content

Commit

Permalink
feat(ingest): Raise ingest error if we ever have multiple loculus acc…
Browse files Browse the repository at this point in the history
…ession for one insdc accession in loculus (#2844)
  • Loading branch information
anna-parker authored Sep 19, 2024
1 parent 767221a commit aff8472
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions ingest/scripts/call_loculus.py
Original file line number Diff line number Diff line change
Expand Up @@ -358,14 +358,13 @@ def get_submitted(config: Config):
"jointAccession": joint_accession,
}
elif loculus_accession != submitted_dict[insdc_accession]["loculus_accession"]:
# For now to be forgiving, just move on, but log the error
# This should not happen in production
message = (
f"INSDC accession {insdc_accession} has multiple loculus accessions: "
f"{loculus_accession} and {submitted_dict[insdc_accession]['loculus_accession']}"
f"{loculus_accession} and "
f"{submitted_dict[insdc_accession]['loculus_accession']}!"
)
logger.error(message)
continue
raise ValueError(message)

submitted_dict[insdc_accession]["versions"].append(
{
Expand Down

0 comments on commit aff8472

Please sign in to comment.