Skip to content

Release 25.08 +SP +SIMD#54

Merged
conda-forge-admin merged 12 commits intoconda-forge:mainfrom
ax3l:release-25.08-sp-simd
Aug 14, 2025
Merged

Release 25.08 +SP +SIMD#54
conda-forge-admin merged 12 commits intoconda-forge:mainfrom
ax3l:release-25.08-sp-simd

Conversation

@ax3l
Copy link
Copy Markdown
Member

@ax3l ax3l commented Aug 13, 2025

Release ImpactX 25.08.

  • Add support for ImpactX_SIMD (disabled for Multipole on windows)
  • Build additional single-precision variants, selectable via conda install -c conda-forge "impactx>=25.08[build=*sp_*]"
    • Default is always double precision (dp)

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@ax3l ax3l added the enhancement New feature or request label Aug 13, 2025
@conda-forge-admin

This comment was marked as resolved.

@conda-forge-admin

This comment was marked as resolved.

@ax3l ax3l force-pushed the release-25.08-sp-simd branch from 4a0b428 to 08bea00 Compare August 13, 2025 17:22
@ax3l
Copy link
Copy Markdown
Member Author

ax3l commented Aug 13, 2025

@conda-forge-admin, please rerender

@conda-forge-admin

This comment was marked as outdated.

@ax3l ax3l force-pushed the release-25.08-sp-simd branch from cb21ce4 to b1d9341 Compare August 13, 2025 18:33
@ax3l
Copy link
Copy Markdown
Member Author

ax3l commented Aug 13, 2025

WeiqunZhang pushed a commit to AMReX-Codes/amrex that referenced this pull request Aug 13, 2025
## Summary

Ensure we can use any producing type's `simd_mask`, e.g., `double`, when
setting (in)valid particles, by manipulating our `uint64` in `idcpu`.
This currently requires a conversion/cast in C++. Make this conversion
portable, so far it only worked on GCC/Linux.

This helper needs C++20. Thus, build AMReX conditionally always with
C++20 or newer when SIMD is requested (non-default). Eventually, we will
require C++26 once this becomes standard, and C++29 will have even more
updates.

## Additional background

Portability to macOS/Windows:
- conda-forge/impactx-feedstock#54
- mattkretz/vir-simd#45

## Checklist

The proposed changes:
- [x] fix a bug or incorrect behavior in AMReX
- [ ] add new capabilities to AMReX
- [ ] changes answers in the test suite to more than roundoff level
- [ ] are likely to significantly affect the results of downstream AMReX
users
- [ ] include documentation in the code and/or rst files, if appropriate
@ax3l ax3l force-pushed the release-25.08-sp-simd branch from 4c73116 to 4ffecf0 Compare August 14, 2025 03:17
@ax3l ax3l force-pushed the release-25.08-sp-simd branch from 4092080 to 76a6a1f Compare August 14, 2025 04:02
@ax3l ax3l force-pushed the release-25.08-sp-simd branch 3 times, most recently from 597e34b to 32f09c7 Compare August 14, 2025 04:38
@ax3l ax3l force-pushed the release-25.08-sp-simd branch from 32f09c7 to ca31784 Compare August 14, 2025 04:49
@ax3l
Copy link
Copy Markdown
Member Author

ax3l commented Aug 14, 2025

Huh, that tolerance should be big enough now o.0

 │ FAILED tests/python/test_charge_deposition.py::test_charge_deposition - assert False
 │  +  where False = <built-in function isclose>(np.float64(-9.999991233739898e-10), -1e-09, rel_tol=7e-07)

Hm, 8.766e-07 is bigger than rel_tol=7e-07

@ax3l ax3l added the automerge Merge the PR when CI passes label Aug 14, 2025
@conda-forge-admin conda-forge-admin merged commit e5e37a9 into conda-forge:main Aug 14, 2025
63 checks passed
@conda-forge-admin
Copy link
Copy Markdown
Contributor

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

  • linter: passed
  • azure: passed

Thus the PR was passing and merged! Have a great day!

@ax3l ax3l deleted the release-25.08-sp-simd branch August 14, 2025 16:56
@ax3l ax3l mentioned this pull request Aug 17, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the PR when CI passes enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants