Skip to content

Fix narrowing conversions and some other minor warnings#252

Merged
jrood-nrel merged 4 commits intodevelopmentfrom
jrood/more-warnings
Apr 15, 2022
Merged

Fix narrowing conversions and some other minor warnings#252
jrood-nrel merged 4 commits intodevelopmentfrom
jrood/more-warnings

Conversation

@jrood-nrel
Copy link
Copy Markdown
Contributor

No description provided.

@jrood-nrel jrood-nrel requested review from esclapez and marchdf April 15, 2022 18:49
@jrood-nrel jrood-nrel merged commit 34442e7 into development Apr 15, 2022
@jrood-nrel jrood-nrel deleted the jrood/more-warnings branch April 15, 2022 19:49
baperry2 added a commit that referenced this pull request Dec 1, 2023
* Explicit filtering of hydro fluxes and source term. (#112)

This commit implements explicit filtering of the hydro fluxes and
source term (for both MOL and PPM code paths). This is necessary to
perform explicitly filtered Large Eddy Simulations.

Contains:
- regressions tests
- documentation
- CMake implementation

* Remove legacy Fortran code. (#252)

* Use default for trivial constructors and only build HIP in CI for gfx908 (#434)

* Use default for trivial constructors.

* Only build HIP in CI for gfx908.

* Update PelePhysics.

* Fixes to header files (#439)

* Fixes to header files.

* Update PelePhysics submodule.

* More fixes.

* Initialize xdiff in HIT.

* Invert some mfiter loops (#446)

Co-authored-by: Jon Rood <[email protected]>

* Add GPU syncs because MF ParallelFor is non-blocking (#471)

* Rename header guards (#506)

* Replace _OPENMP with AMREX_USE_OMP to support Ascent. (#519)

* Address some warnings (#554)

* remove dependence of Filter on Utilities (#710)

* move files

* docs for filter

* use amrex Pi instead of PeleC Pi

---------

Co-authored-by: Marc T. Henry de Frahan <[email protected]>
Co-authored-by: Jon Rood <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants