Skip to content

acts: backport Gen3 DD4hep construction#887

Merged
wdconinc merged 2 commits into
developfrom
acts-gen3
Apr 19, 2026
Merged

acts: backport Gen3 DD4hep construction#887
wdconinc merged 2 commits into
developfrom
acts-gen3

Conversation

@wdconinc

@wdconinc wdconinc commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

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

This PR backports the Acts Gen3 DD4hep construction acts-project/acts#5193 into acts v45 so we can roll this out in development sooner.

Tested that this cherry-picks cleanly into 45.2.0, and that it is included in release 46.3.0. Checked with AI that there are no hidden dependencies that would affect functionality outside of cherry-pick merge conflicts.

What is the urgency of this PR?

  • High
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue: Acts Gen3 DD4hep construction)
  • 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 AI review requested due to automatic review settings April 18, 2026 16:52
@wdconinc wdconinc requested a review from veprbl April 18, 2026 16:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Backports the Acts “Gen3 DD4hep construction” change into the Acts v45 Spack package by applying the upstream patch from acts-project/acts#5193.

Changes:

  • Add a new Spack patch() entry to apply the Gen3 DD4hep construction backport for Acts versions @45.2:46.2.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread spack_repo/eic/packages/acts/package.py Outdated
@wdconinc wdconinc enabled auto-merge (squash) April 18, 2026 16:56
wdconinc added a commit that referenced this pull request Apr 18, 2026
Spack switched to ruff for style checks. This adds a minimal
`pyproject.toml` to configure ruff so that CI style checks pass.

The key fix is the `per-file-ignores` section that suppresses `F403`,
`F405`, `F811`, and `F821` for package files, which use `from
spack.package import *` (star imports are expected in Spack packages).

Fixes the CI failure in #887.

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@wdconinc wdconinc merged commit 8649fb5 into develop Apr 19, 2026
6 checks passed
@wdconinc wdconinc deleted the acts-gen3 branch April 19, 2026 20:58
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.

3 participants