Skip to content

Refactor reporting + timeseries functionality

Patrick Godwin requested to merge refactor_reports into master

This PR restructures how reports are generated so that individual pieces (generating plots, discovering data) have been decoupled from the Report class, allowing customization for other external programs like DQR tasks. To facilitate this, the timeseries handling throughout has also been refactored so instead of passing around a nested dict assuming a specific structure throughout, they're now wrapped around several lightweight dataclasses to make this easier to work with.

Some extra things caught as part of this PR:

  • Remove conditional logger in Reports. This was removed in other places but not the Report, so here it's finally done.
  • Remove unused html import in idq.reports.
  • Remove problematic --preferred options in various programs as the individual batch programs don't make use of this anymore. It could be added back in in the future, but led to confusing behavior without much usage.

Merge request reports