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

build: Improve traccc dependency loading #3478

Merged

Conversation

stephenswat
Copy link
Member

@stephenswat stephenswat commented Aug 5, 2024

This commit makes a few changes to the way some of the Acts R&D projects are loaded in Acts (i.e. detray, traccc, algebra-plugins), namely:

  1. It adds code to load detray's dependencies if the Acts project is installed.
  2. It updates the versions of detray and traccc so they are compatible.
  3. It adds some documentation to the thirdparty/ directory.
  4. It refines the parameters for the CMake build systems of various projects to be more robust.

This commit comes from #3117.

@github-actions github-actions bot added the Infrastructure Changes to build tools, continous integration, ... label Aug 5, 2024
@stephenswat
Copy link
Member Author

Actually, this needs more integration with the _acts_XXXX_version mechanism.

@stephenswat stephenswat force-pushed the build/update_traccc_externals branch from 0cc148a to bffa47c Compare August 5, 2024 13:16
@stephenswat
Copy link
Member Author

Okay, this should now be good to go.

Copy link
Contributor

@andiwand andiwand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@stephenswat stephenswat force-pushed the build/update_traccc_externals branch 2 times, most recently from d991cd1 to f291f92 Compare August 5, 2024 13:40
This commit makes a few changes to the way some of the Acts R&D projects
are loaded in Acts (i.e. detray, traccc, algebra-plugins), namely:

1. It adds code to load detray's dependencies if the Acts project is
   installed.
2. It updates the versions of detray and traccc so they are compatible.
3. It adds some documentation to the `thirdparty/` directory.
4. It refines the parameters for the CMake build systems of various
   projects to be more robust.
5. It establishes a dependency of the traccc plugin on the detray
   plugin, and the detray plugin on the covfie plugin.

Co-authored-by: Stephen Nicholas Swatman <[email protected]>
@github-actions github-actions bot added the Component - Documentation Affects the documentation label Aug 5, 2024
@andiwand andiwand added this to the next milestone Aug 5, 2024
Copy link

sonarqubecloud bot commented Aug 5, 2024

@stephenswat
Copy link
Member Author

@andiwand any idea why policy-bot is refusing to accept this?

@andiwand andiwand removed the automerge label Aug 6, 2024
@andiwand
Copy link
Contributor

andiwand commented Aug 6, 2024

looks like I didn't approve 😅

@kodiakhq kodiakhq bot merged commit 0c37673 into acts-project:main Aug 6, 2024
49 checks passed
@github-actions github-actions bot removed the automerge label Aug 6, 2024
@acts-project-service
Copy link
Collaborator

acts-project-service commented Aug 6, 2024

🔴 Athena integration test results

Build job with this PR failed!

Please investigate the build job for the pipeline!

@acts-project-service acts-project-service added the Breaks Athena build This PR breaks the Athena build label Aug 6, 2024
@paulgessinger paulgessinger modified the milestones: next, v36.1.0 Aug 19, 2024
kodiakhq bot pushed a commit that referenced this pull request Aug 29, 2024
This commit adds a new Acts plugin that adds support for magnetic fields implemented using the covfie library.

This commit is based on #3117.

Closes #3117.

Depends on #3478.

Virtually all credit goes to @fredevb.

Co-authored-by: Fred <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaks Athena build This PR breaks the Athena build Component - Documentation Affects the documentation Infrastructure Changes to build tools, continous integration, ...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants