-
Notifications
You must be signed in to change notification settings - Fork 0
Pass 3 step 1 GCD audits #27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@gh999ic Please check that the "SLC only" doms are not lumped into the general bad doms list. They currently seem to have mean_fadc_charge = nan in 2024 run DB. I suspect they've always been this way here's a list I just check for a 2024 run: 140733. OMKey(7,34,0) SLC only |
how can I make a list like that? |
this is just a list of DOMs that have nan values in mean_fadc_charge. I then did a dive into live.icecube.wisc.edu to see why these doms would not have these values. These SLC only doms were ignored by Dawn's calibration scripts... |
I originally ran the I3GCDAuditor module which omits checking any doms Then I made the following changes to BadDomListModule, I3GCDAuditor, and gcd_generation in PR #3892: The BadDomListModule could already accept a list of custom OMKeys, In I3GCDAuditor, I added info and warn msgs if a DOM has a nan value for mean ATWD or FADC charge. In gcd_generation, I added functions run_gcd_audit_pass3 and get_nan_doms. The full workflow is in PR #28
Does this sound reasonable? I already used all of the above to create a new set of historical GCD files at |
These audits should check at all DOMs have are working properly have a value of 1.0 for mean_atwd_charge and mean_fadc_charge
The text was updated successfully, but these errors were encountered: