Skip to content
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

ci: Update LCG tests #2410

Merged
merged 20 commits into from
Sep 29, 2023
Merged

ci: Update LCG tests #2410

merged 20 commits into from
Sep 29, 2023

Conversation

paulgessinger
Copy link
Member

@paulgessinger paulgessinger commented Aug 30, 2023

Removes the forces assertions from the LCG builds, because I've observed these to fail in clang15 on LCG 103.
I don't think we necessarily need to have these turned on, and we know this is a hack anyway.

This PR then also includes builds for LCG104, and switches the LCG102 build over to LCG102b_ATLAS_23 (which is what Athena currently builds on)

Blocked by:

This showed up in our GCC12 based LCG builds, and only in Release mode.
I don't think there's a way for us to fix this because the warning
actually comes from `std::move`'ing `boost::small_vectors`. I'm
suppressing the warnings here and remove the job from being allowed to
fail.
@github-actions github-actions bot added Component - Core Affects the Core module Seeding labels Aug 30, 2023
@paulgessinger paulgessinger added the 🛑 blocked This item is blocked by another item label Aug 30, 2023
@codecov
Copy link

codecov bot commented Aug 30, 2023

Codecov Report

Merging #2410 (db72ac1) into main (4a4f858) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2410   +/-   ##
=======================================
  Coverage   49.82%   49.82%           
=======================================
  Files         466      466           
  Lines       26246    26247    +1     
  Branches    12050    12050           
=======================================
+ Hits        13077    13078    +1     
  Misses       4603     4603           
  Partials     8566     8566           
Files Coverage Δ
...de/Acts/TrackFinding/CombinatorialKalmanFilter.hpp 32.10% <100.00%> (+0.15%) ⬆️
Core/src/Geometry/GenericCuboidVolumeBounds.cpp 33.61% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@paulgessinger
Copy link
Member Author

Ok:

  • clang12 is broken as expected
  • Seems I missed a problem in gcc12/13
  • gcc11 on LCG_102b_ATLAS_23 has a git clone issue of all things?
    To be investigated.

@paulgessinger paulgessinger marked this pull request as draft August 30, 2023 15:53
@github-actions github-actions bot removed Component - Core Affects the Core module Seeding labels Sep 5, 2023
@paulgessinger paulgessinger added this to the next milestone Sep 5, 2023
@paulgessinger paulgessinger removed the 🛑 blocked This item is blocked by another item label Sep 5, 2023
@github-actions github-actions bot added Component - Core Affects the Core module Component - Examples Affects the Examples module labels Sep 11, 2023
@github-actions github-actions bot added Component - Plugins Affects one or more Plugins Track Finding labels Sep 12, 2023
@github-actions github-actions bot added the Component - Fatras Affects the Fatras module label Sep 12, 2023
@paulgessinger paulgessinger marked this pull request as ready for review September 28, 2023 15:34
@kodiakhq kodiakhq bot merged commit 4515f2c into acts-project:main Sep 29, 2023
58 of 59 checks passed
AJPfleger pushed a commit to AJPfleger/acts that referenced this pull request Sep 29, 2023
Removes the forces assertions from the LCG builds, because I've observed these to fail in clang15 on LCG 103.
I don't think we necessarily need to have these turned on, and we know this is a hack anyway.

This PR then also includes builds for LCG104, and switches the LCG102 build over to LCG102b_ATLAS_23 (which is what Athena currently builds on)

Blocked by:
- acts-project#2407
@paulgessinger paulgessinger modified the milestones: next, v30.1.0 Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Core Affects the Core module Component - Examples Affects the Examples module Component - Fatras Affects the Fatras module Component - Plugins Affects one or more Plugins Track Finding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants