AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-manager

Quality control write_standard_file() will overwrite existing metadata

Aperta
#33 0 commenti 0 reazioni 1 assegnatario Rivendicata da @seanmcculloch Vedi su GitHub
Lingua principale
Python
Stelle
0
Fork
0
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

**Describe the bug**
At [this line](https://github.com/AllenNeuralDynamics/aind-metadata-manager/blob/11e9f2fa2b0be6bb84cceda6976dcc5b3e19f880/src/aind_metadata_manager/metadata_manager.py#L550) the existing manager code calls `write_standard_file()` for the QC metadata. If a previous quality_control.json file existed, it will be overwritten at this step.

**Expected behavior**
If one exists, a previous quality_control.json file should be loaded, validated as a `QualityControl` object and then merged with the new QC metrics by calling `qc = qc_old + qc_new`. Then `qc.write_standard_file()` on the final merged output.

**Other context**
This was also happening for processing metadata, but I flagged this to Sean in a separate PR and it sounds like he'll fix it there.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.