AllenInstitute / brain_observatory_qc

Other
2 stars 0 forks source link

Re-evaluate intensity drift (%) metric #356

Open jkim0731 opened 8 months ago

jkim0731 commented 8 months ago

Describe the Issue

Screenshots image image

https://alleninstitute-my.sharepoint.com/:p:/g/personal/jinho_kim_alleninstitute_org/EeRvwo3hEB9Lro1fVVH1tJIBbYvum2CtDe1yG6SMlnAZGA?e=lmDFor

Scope

jkim0731 commented 8 months ago
seanmcculloch commented 8 months ago

For now, Operator's should use the "other" tag to flag the fluorescence plot. During our review of "Other" tags, we can create meaningful controlled language tags around various failures, and potentially update the intensity drift metric (or another metric) to be more comprehensive

DowntonCrabby commented 7 months ago

the % change currently checks the first portion of the timeseries to the last portion of the timeseries- so you're correct in that it won't catch any intermediate drastic changes.

Maybe we can make some automated metrics that would be able to catch this- like a sliding window , or standard deviation. What do you think @jkim0731

samiamseid commented 7 months ago

connected to following issues:

317

188

67