Skip to content

Modify Reporters API, cleanup

Patrick Godwin requested to merge reporters_update into main

This PR focuses on Reporters generally, with the major change being that Reporter.read is now part of the public API.

This should allow anyone to be able to read in data products generated by iDQ given just a path to the data product in question, e.g.data = DiskReporter.read(path), without needing specifics like the rootdir when instantiating a Reporter.

Other changes:

  • Use os.makedirs(directory, exist_ok=True) throughout.
  • Flatten conditional in DiskReporter's glob implementation.
  • Remove atomic FIXMEs, as all these have been addressed by writing to a temporary file and a move.

Merge request reports