V2.2.02

[glesur]

Changed

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

Added

• documentation for the continuous integration (Doc updates #354)

[Copilot]

Pull Request Overview

This pull request upgrades Idefix to version 2.2.02, incorporating several bug fixes and a new units module for explicit unit conversions. The changes address critical issues in profiling, energy budget calculations, boundary conditions, and magnetic field reconstruction while adding comprehensive documentation for continuous integration testing.

Key changes:

• Added a units module that provides explicit unit conversion capabilities in CGS system
• Fixed multiple bugs related to profiling, energy budget, boundary conditions, and dust physics
• Enhanced documentation for continuous integration testing workflows

Reviewed Changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
src/units.hpp	New header defining the Units class with CGS constants and unit conversion methods
src/units.cpp	Implementation of Units class methods for initialization and unit calculations
src/main.cpp	Integration of units module into main application workflow
src/gravity/gravity.cpp	Enhanced gravity module to support unit-based gravitational constant calculation
src/global.hpp/cpp	Added global units object and profiling fence fix
src/fluid/drag.hpp	Fixed parentheses placement in dust stopping time calculation
src/fluid/calcRightHandSide.hpp	Corrected energy budget calculation for Fargo+shearing box scenarios
src/fluid/boundary/boundary.hpp	Improved BX2 reconstruction and reflective boundary condition handling
src/fluid/boundary/axis.hpp/cpp	Refactored axis boundary handling and removed debug output
src/fluid/addSourceTerms.hpp	Removed duplicate Fargo source term causing energy budget issues
doc/source/*	Added comprehensive CI testing documentation
CMakeLists.txt	Updated version numbers and build configuration

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

src/fluid/boundary/axis.hpp:1

• Removed debug output statement that was likely left over from development. This change improves code cleanliness by removing unnecessary console output.

// ***********************************************************************************

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}