Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .binder/environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@ dependencies:
- coveralls
- coverage
- setuptools
- numpy =2.3.5
- numpy =2.4.3
- black =25.1.0
- h5py =3.16.0
Comment on lines +7 to 9
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

NumPy is pinned here to 2.4.0, but pyproject.toml pins 2.4.3 (and CI may use a different pin). Please keep the Binder environment aligned with the version used for packaging/CI to reduce "works in one env but not another" issues.

Copilot uses AI. Check for mistakes.
- ase =3.28.0
- sphinxdft-data =0.0.1
- semantikon =1.0.0
- pint =0.25.2
- pint =0.25.3
- pyyaml =6.0.3
4 changes: 2 additions & 2 deletions .ci_support/environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@ dependencies:
- coveralls
- coverage
- setuptools
- numpy =2.3.5
- numpy =2.4.3
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

This updates NumPy to 2.4.3, but other env definitions (e.g., docs and Binder) pin NumPy to 2.4.0. Please standardize on a single NumPy patch version across all environment files and pyproject.toml (or document why they intentionally differ), otherwise debugging CI vs docs/Binder failures becomes harder.

Suggested change
- numpy =2.4.3
- numpy =2.4.0

Copilot uses AI. Check for mistakes.
- black =25.1.0
- h5py =3.16.0
- ase =3.28.0
- sphinxdft-data =0.0.1
- semantikon =1.0.0
- pint =0.25.2
- pint =0.25.3
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

Pint is bumped here to 0.25.3, but some other conda env files still pin Pint to 0.25.2. Please align Pint pins across environments (or add an explicit reason for the divergence) to avoid subtle unit-handling differences between CI and docs/Binder.

Suggested change
- pint =0.25.3
- pint =0.25.2

Copilot uses AI. Check for mistakes.
- pyyaml =6.0.3
4 changes: 2 additions & 2 deletions docs/environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ dependencies:
- coveralls
- coverage
- setuptools
- numpy =2.3.5
- numpy =2.4.3
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

Along with the NumPy bump, this environment remains on Pint 0.25.2 while pyproject.toml has Pint 0.25.3. If docs builds are intended to match the packaged dependency set, consider bumping Pint here too (or documenting why it must remain older).

Copilot uses AI. Check for mistakes.
- black =25.1.0
- h5py =3.16.0
Comment on lines +13 to 15
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

NumPy is pinned here to 2.4.0, but pyproject.toml pins NumPy to 2.4.3. Please keep the pins consistent across packaging and documentation environments (or explicitly call out that docs build uses a different NumPy) to prevent hard-to-reproduce doc build issues.

Copilot uses AI. Check for mistakes.
- ase =3.28.0
- sphinxdft-data =0.0.1
- semantikon =1.0.0
- pint =0.25.2
- pint =0.25.3
- pyyaml =6.0.3
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@ classifiers = [
"Programming Language :: Python :: 3.14",
]
dependencies = [
"numpy==2.3.5",
"numpy==2.4.3",
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

PR title says updating NumPy to 2.4.0, but this file pins NumPy to 2.4.3. Please align the intended NumPy version across the repo (and update the PR title/description if 2.4.3 is intended) to avoid confusing or divergent dependency resolution between packaging and conda envs.

Copilot uses AI. Check for mistakes.
"ase==3.28.0",
"h5py==3.16.0",
"semantikon==1.0.0",
"pint==0.25.2",
"pint==0.25.3",
"pyyaml==6.0.3",
Comment on lines 29 to 31
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

This bumps Pint to 0.25.3 for the package, but some conda environment files still pin Pint to 0.25.2. That can lead to behavior differences between pip installs and docs/CI/Binder environments; consider updating all environment.yml pins to the same Pint version (or documenting why they must differ).

Copilot uses AI. Check for mistakes.
]
dynamic = [ "version",]
Expand Down
Loading