Skip to content

feat: g4adept-0.3.0 in CUDA containers#235

Open
wdconinc wants to merge 7 commits intomasterfrom
g4adept
Open

feat: g4adept-0.3.0 in CUDA containers#235
wdconinc wants to merge 7 commits intomasterfrom
g4adept

Conversation

@wdconinc
Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

This PR installs AdePT in our cuda containers.

What is the urgency of this PR?

  • High
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue: GPU acceleration of simulations)
  • Optimization (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

Copilot suggested a commit message. This can't be turned off.

Copilot AI review requested due to automatic review settings April 12, 2026 14:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to add the AdePT (g4adept) package into the CUDA container build by bringing in the necessary Spack package definitions via spack-packages cherry-picks.

Changes:

  • Added spack-packages cherry-picks for newly introduced g4hepem and g4adept packages.
  • Documented the new cherry-picks in the hash reference comment block.

Copilot AI review requested due to automatic review settings April 12, 2026 14:12
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

spack-environment/packages.yaml:196

  • +autotools is not used anywhere else in packages.yaml, and Spack typically selects the build system via build_system=autotools (see freetype at spack-environment/packages.yaml:183-186). As written, +autotools is likely an invalid variant for gdb and may break concretization; consider restoring the previous +debuginfod requirement or switching to build_system=autotools if that was the intent.
  gdb:
    require:
    - '@15.2:'
    - +debuginfod

Copilot AI review requested due to automatic review settings April 12, 2026 14:16
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants