-
Notifications
You must be signed in to change notification settings - Fork 483
DPL: add possibility to disable downscaling of processing reporting by env variable #14656
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
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
ea71866 to
1b0694d
Compare
|
Error while checking build/O2/fullCI_slc9 for 1b0694d at 2025-09-12 04:01: Full log here. |
| if (!prescale) { | ||
| prescale = 1; | ||
| } | ||
| bool noDownscaling = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you make this static, so that we do not always query the environment:
static bool noDownscaling = getenv("DPL_REPORT_PROCESSING_NO_DOWNSCALING") != nullptr && std::abs(atoi(getenv("DPL_REPORT_PROCESSING_NO_DOWNSCALING")))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, done
|
Error while checking build/O2/fullCI_slc9 for 0fdbed0 at 2025-09-22 03:43: Full log here. |
|
@singiamtel the issue with the failing CCDB tests still seems to persist. @ktf do/can you approve the PR? |
No description provided.