Skip to content

fix incorrect subviews when using vector_potential in 2.5D#362

Merged
glesur merged 2 commits into
developfrom
fixVectorPotential2D
Jan 21, 2026
Merged

fix incorrect subviews when using vector_potential in 2.5D#362
glesur merged 2 commits into
developfrom
fixVectorPotential2D

Conversation

@glesur
Copy link
Copy Markdown
Contributor

@glesur glesur commented Jan 20, 2026

fix a bug that led to the generation of incorrect subviews in 2.5D with vector_potential enabled. Note that this bug was silent, and is detected only when Kokkos_DEBUG is enabled.

@glesur glesur added the bug Something isn't working label Jan 20, 2026
@glesur glesur merged commit d83d694 into develop Jan 21, 2026
38 checks passed
@glesur glesur deleted the fixVectorPotential2D branch January 21, 2026 09:21
@glesur glesur mentioned this pull request Apr 21, 2026
glesur added a commit that referenced this pull request Apr 22, 2026
### Changed

- fixed a bug that could lead to diverging results around the spherical axis in non-ideal MHD in 2.5D and 3D (#356)
- refactor of the MPI exchange routines and boundary routines to avoid buildup of roundoff errors at domain faces/edges that could lead to the sudden burst of div(B) or incoherences between MPI sub-domains (#357)
- fixed a bug that could lead to compilations error when targetting AMD APUs (#359)
- fixed a bug that led to the generation of incorrect subviews in 2.5D with vector_potential enabled (#362)
- fixed a bug that could lead to memory corruption when using the UCT_HLLD emf reconstruction scheme and DIMENSIONS < COMPONENTS  (#363)
- reorganise the test to separate specific IO tests from physics tests (#367)

### Added
- magnetic vector potential is now accessible from pydefix when enabled (#361)
- use ccache in the test suite to reduce the runtime of continuous integration (#364)
- automatically detects pybind11 path with cmake when using pydefix (#367)
- skeleton to run the test suite with pytest (#366)
---------------------------------------------------------------------------
Co-authored-by: Clément Robert <cr52@protonmail.com>
Co-authored-by: Sébastien Valat <sebastien.valat.dev@orange.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant