-
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
fix: missing include of array in FpeMonitor.hpp #2247
Conversation
📊 Physics performance monitoring for 9ee1d22Summary VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
#2248 should catch such issues in the future (at least for |
Codecov Report
@@ Coverage Diff @@
## main #2247 +/- ##
=======================================
Coverage 49.35% 49.35%
=======================================
Files 445 445
Lines 25264 25264
Branches 11650 11650
=======================================
Hits 12468 12468
Misses 4512 4512
Partials 8284 8284 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
I keep finding this kind of compilation issues...
This adds a simple script to check all main header files of ACTS for missing includes (like in #2247). This is done very simple: We try to compile (and not link) ```c++ #include <file> int main() { return 0; } ``` If this succeeds, the compiler is able to parse the whole file sucessfully, and thus no include is missing. This check takes about 10 minutes on my machine for the `Core` package... I think that is worth it.
This adds a simple script to check all main header files of ACTS for missing includes (like in acts-project#2247). This is done very simple: We try to compile (and not link) ```c++ #include <file> int main() { return 0; } ``` If this succeeds, the compiler is able to parse the whole file sucessfully, and thus no include is missing. This check takes about 10 minutes on my machine for the `Core` package... I think that is worth it.
I keep finding this kind of compilation issues...