FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix profiling on GPUs by glesur · Pull Request #341 · idefix-code/idefix · GitHub

Repository navigation

fix profiling on GPUs - #341

Merged
glesur merged 1 commit into
developfrom
fixProfilingGPU
May 16, 2025
Merged

glesur merged 1 commit into
developfrom
fixProfilingGPU

Conversation

glesur commented May 16, 2025

Copy link
Copy Markdown
Contributor

fix #339

Add Kokkos::fence at the beginning of profiling regions to fix profiling incorrect timing on GPUs

glesur added the bug Something isn't working label May 16, 2025
glesur merged commit e4b5a83 into develop May 16, 2025
glesur deleted the fixProfilingGPU branch May 16, 2025 16:30
glesur mentioned this pull request Oct 18, 2025
glesur added a commit that referenced this pull request Oct 19, 2025
## [2.2.02] 2025-10-18
### Changed

- Added a module to add explicit units (#338)
- fixed a bug that could lead to incorrect profiling information on non-blocking cuda loops (#341)
- fixed a bug that could lead to incorrect energy budget when shearing box and fargo were both enabled (#346)
- fixed a bug that led to incorrect BX2 reconstruction when axis is not used on both sides of the domain (#345)
- fixed a bug that led to incorrect reflective boundary conditions on B when DIMENSIONS < 3 (#345)
- fixed a bug that led to incorrect dust stopping time when the adiabatic equation of state is used with "size" drag law (#353)

### Added

- documentation for the continuous integration (#354)
---------

Co-authored-by: Victor Réville <victorreville@gmail.com>
Co-authored-by: Hal Bal <vreville@irap.omp.eu>
Co-authored-by: Jean Kempf <jean.kempf@irap.omp.eu>
Co-authored-by: Victor Réville <47865059+vreville@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL