Closed steven-murray closed 1 year ago
Patch coverage: 100.00
% and no project coverage change.
Comparison is base (
c34b82e
) 95.98% compared to head (032bd55
) 95.98%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Ah, I should have linked to the PR in hera_cal: https://github.com/HERA-Team/hera_cal/pull/861#issuecomment-1382451023.
Basically, there's a new module in hera_cal
that adds parameters to a CLI script for controlling logging and line-profiling. These are super helpful for performance testing. It's opt-in, so doesn't change anything for anyone by default. If turned on, it'll log the memory usage at each logging statement, looks like this:
Perfect! thanks for the explanation @steven-murray and for updating the calls to merge_spectra
functions.
I'll merge and closer the pull request now.
This simply uses the new cli-tools in hera_cal to add a bit of functionality to hera_pspec scripts.