Closed OliviaLynn closed 9 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
263b2a6
) 96.96% compared to head (e125207
) 96.96%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Problem & Solution Description (including issue #)
For the bpz portion of rail#48.
I wasn't 100% sure if we wanted to keep the other (non-tar) data separate like the tar data.
If downloading
test_dc2_training_9816_broadtypes.hdf5
andtest_dc2_train_customtemp_broadttypes.hdf5
from NERSC along with the tar works for people, here's a PR for that.This PR also reduces the duplicated logic across the if/else branching.
Code Quality
#pragma: no cover
; in the case of a bugfix, a new test that breaks as a result of the bug has been added