-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add scalar floor and refactor temperature and density floors #363
Merged
evaneschneider
merged 59 commits into
cholla-hydro:dev
from
helenarichie:dev-scalar-floor
Feb 1, 2024
Merged
Changes from 46 commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
3ecb87d
add function to apply a floor to any conserved variable and apply a d…
helenarichie 3dd1d30
remove unnecessary macro
helenarichie 198200b
Merge branch 'dev' into dev-scalar-floor
helenarichie 2a6eb5f
save progress
helenarichie c793c79
Merge branch 'dev' into dev-scalar-floor
helenarichie 4fda7e2
Merge branch 'dev-scalar-floor' of github.com:helenarichie/cholla int…
helenarichie 9a245c3
Merge branch 'dev' into dev-scalar-floor
helenarichie b341db6
Merge branch 'dev-scalar-floor' of github.com:helenarichie/cholla int…
helenarichie 8a58a85
Merge branch 'dev' into dev-scalar-floor
helenarichie 25832d4
Merge branch 'dev' into dev-scalar-floor
helenarichie 36a54e3
Merge branch 'dev-scalar-floor' of github.com:helenarichie/cholla int…
helenarichie 77d6e1a
add test for scalar floor kernel
helenarichie 639be11
run clang format
helenarichie 25cdde9
Merge branch 'dev' into dev-scalar-floor
helenarichie 0fd7f89
update branch
helenarichie 098fe22
update initial conditions
helenarichie 9b7e614
save simulation setup
helenarichie fed86f7
resolve merge conflict
helenarichie 4077edd
resolve merge conflict
helenarichie 3cd6b2e
update from cell averaging PR
helenarichie c1ccee3
update branch
ed40066
Merge branch 'dev' into dev-scalar-floor
5b85251
update build
f5dc360
hard-code cloud temp
helenarichie 60b51a5
Merge branch 'dev-scalar-floor' of github.com:helenarichie/cholla int…
helenarichie e12b846
update build
helenarichie 3fec978
update build
helenarichie 328770b
clean up initial conditions
helenarichie 2f3ef5f
reset to default settings
helenarichie 094f1e9
get floor values from paramter file and remove density floor kernel
helenarichie d783bb7
run clang format
helenarichie 399921b
remove reference to DENS_FLOOR macro
helenarichie fa7cfba
Merge branch 'dev' into dev-scalar-floor
helenarichie f276258
run clang format
helenarichie 37ccafd
run clang format
helenarichie d737c4b
run clang format
helenarichie 5f46029
Merge branch 'dev-scalar-floor' of github.com:helenarichie/cholla int…
helenarichie d292c46
undo accidental change
helenarichie da2d264
undo accidental removal of density floor in half-step
helenarichie 8945bd4
Merge branch 'dev-scalar-floor' of github.com:helenarichie/cholla int…
helenarichie ef47030
fix undeclared variable
helenarichie 2a5d7ed
remove debugging print statement
helenarichie db0f788
add host wrapper functions for temp and scalar floor kernels and move…
helenarichie 71df115
run clang format'
helenarichie cfd9c0b
Merge branch 'dev' into dev-scalar-floor
helenarichie cb5e8f3
run clang format
helenarichie ddbd312
Update hydro build to use VL and PLMC
bcaddy 29aa051
Switch MHD to using PLMC by default
bcaddy fafcf4f
Update other builds to PLMC and VL default
bcaddy 2215b2f
Make sure cosmology builds use SIMPLE integrator
bcaddy ff09d5d
Fix clang-tidy warning
bcaddy b458962
Replace #warning with print to cerr
bcaddy 35e327a
Merge branch 'dev' into dev-scalar-floor
helenarichie 3e1c195
set default floor values in global.h and add warning
helenarichie 8e2c742
Merge branch 'dev' into dev-scalar-floor
helenarichie 8610f35
Merge branch 'dev' into dev-scalar-floor
helenarichie 114f96b
change warning message and tweak how the default floors are set
helenarichie 3eb2fb8
fix syntax errors
helenarichie cede7f3
run clang format
helenarichie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While I understand this logic for the density and temperature floors, I think it doesn't actually apply for the scalar floor, since scalars can have negative values (think for example about the transverse velocities in the Riemann problem). In practice, it shouldn't matter, since you don't call the floor function unless you are applying a floor, in which case the value should have been specified in the parameter file. But I think it might make more sense rather than setting a default value of 0 here to just have the code throw a warning if the floor macro is turned on but no input value is supplied.