Skip to content

chore: edm4hep-1.0, podio-1.7#165

Open
wdconinc wants to merge 5 commits intomasterfrom
edm4hep-1.0
Open

chore: edm4hep-1.0, podio-1.7#165
wdconinc wants to merge 5 commits intomasterfrom
edm4hep-1.0

Conversation

@wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Feb 16, 2026

Briefly, what does this PR introduce?

Needs:

This PR upgrades edm4hep to 1.0 and podio to 1.7.

Copilot AI review requested due to automatic review settings February 16, 2026 20:08
Copy link
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 upgrades two key HEP data model packages: edm4hep from 0.99.4 to 1.0 and podio from 1.6 to 1.7. The changes incorporate upstream Spack package definitions through cherry-picked commits and update the default version constraints in the build environment configuration.

Changes:

  • Add three cherry-pick commits from spack/spack-packages to incorporate new package versions and python freethreading conflict constraints
  • Update edm4hep version requirement from '@0.99.4' to '@1.0' in packages.yaml
  • Update podio version requirement from '@1.6' to '@1.7' in packages.yaml

Reviewed changes

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

File Description
spack-packages.sh Adds three cherry-pick commits (7e4068a, 208b3c4, 9ba7aca) with proper documentation for freethreading conflicts, podio 1.7, and edm4hep 1.0
spack-environment/packages.yaml Updates version constraints for edm4hep and podio packages to match the new versions

Copilot AI review requested due to automatic review settings February 16, 2026 22:57
Copy link
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