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.

Sign up for GitHub

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

Jump to bottom

Correctly use the floor instead of rounding to determine gauge cell index #732

Merged

ketch merged 1 commit into clawpack:master from mandli:fix-gauge-rounding

Oct 6, 2024

Member

mandli commented Oct 5, 2024

This replaces #730 with a quick fix for an extraneous parenthesis. Original PR -

Thanks to Jerry Fox for catching this: https://groups.google.com/g/claw-users/c/5da9VH_3aAc


          Add correct fix for gauge index rounding

af04db2

Member

ketch commented Oct 6, 2024

Oof, sorry for that and thanks for catching it!

ketch merged commit b049ac4 into clawpack:master

1 of 2 checks passed

ketch deleted the fix-gauge-rounding branch

October 6, 2024 04:13

rjleveque mentioned this pull request

examples/shallow_sphere gives segmentation fault #642

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet