-
Notifications
You must be signed in to change notification settings - Fork 20
DQM optimization #203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DQM optimization #203
Conversation
…nly the output FITS results file is relevant).
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #203 +/- ##
==========================================
+ Coverage 52.25% 53.36% +1.11%
==========================================
Files 76 76
Lines 6329 6234 -95
==========================================
+ Hits 3307 3327 +20
+ Misses 3022 2907 -115 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…out the DQM, to speed things up and avoid unpacking unnecessary FEB data
…local mode from a different directory than this script
|
In the end, the speed-up factor is about 4-5, and memory consumption is reduced by a factor ~8 on large runs. |
…ally when dealing with large amounts of data in one plot.
…ther DQM classes.
This PR proposes to clean, streamline and optimize the DQM code. At the moment, I benchmarked a speed-up of a factor 3 on the processing time.