-
Notifications
You must be signed in to change notification settings - Fork 157
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
feat: Add ability to mask FPEs from code comment #2277
feat: Add ability to mask FPEs from code comment #2277
Conversation
Syntax is ```cpp float v = 1.0 / 0.0; // MARK: fpeMask(FLTDIV, 1) ```
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.
nice! does this also work for long code lines? the formatter might insert line breaks as well
@andiwand You need to handle this manually, to ensure the comment is on the exact line. There are no other smarts applied. |
Had to fix the format, can you reapprove @andiwand ? |
Codecov Report
@@ Coverage Diff @@
## main #2277 +/- ##
==========================================
- Coverage 49.27% 49.27% -0.01%
==========================================
Files 450 450
Lines 25410 25415 +5
Branches 11724 11727 +3
==========================================
+ Hits 12521 12523 +2
+ Misses 4556 4554 -2
- Partials 8333 8338 +5 see 9 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for 94cb805Summary VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
Syntax is ```cpp float v = 1.0 / 0.0; // MARK: fpeMask(FLTDIV, 1) ``` where you can specify the FPE type, and the maximum number per event that is tolerated. You can add multiple `fpeMask()` expressions per line. NOTE: This will likely only work in case of running from the build directory, not from an installed one.
Syntax is
where you can specify the FPE type, and the maximum number per event that is tolerated. You can add multiple
fpeMask()
expressions per line.NOTE: This will likely only work in case of running from the build directory, not from an installed one.