Skip to content

Bump release to 24.2.13, fix some bugs

Dan Guest requested to merge dguest/hh4b-analysis:bump into master

This required a few unexpected changes:

  • For some reason the eventNumber is now being accessed somewhere as unsigned long, whereas before I was reading it as unsigned long long. This only broke the H5 writer which is more finicky about type, I put in a patch that keeps the same type.
  • Something in the metadata became impossible to serialize in json. I added a fallback to pickle.

I also found some issues that were probably introduced with refactoring:

  • We were missing a unit test for a more full analysis in data. I added a unit test for sh4b in data.
  • Rerunning a job with the --config-only option would crash because the function that exited early in this case had the wrong return type. I made it abort the job instead with exit.
Edited by Dan Guest

Merge request reports