Correct the use of rshift to reduce the number of distance calculations. (!403) · Merge requests · SWIFT / SWIFTsim · GitLab

Snippets Groups Projects

Merged James Willis requested to merge fix_rshift_bug into master 7 years ago

Applies rshift (boundary conditions) correctly in DOPAIR1 + DOPAIR2 to reduce the number of spurious distance calculations.

Activity

James Willis mentioned in issue #313 (closed) 7 years ago

mentioned in issue #313 (closed)
Matthieu Schaller @matthieu 7 years ago

Owner

Can I dare ask whether the EAGLE_25 time-step sizes are identical in both cases ?
James Willis @jwillis 7 years ago

Author Guest

Haha way ahead of you. I always run it now, they're fine
Matthieu Schaller @matthieu 7 years ago

Owner

Ok. That's good to go then.
Matthieu Schaller Status changed to merged 7 years ago

Status changed to merged
Matthieu Schaller mentioned in commit fca42236 7 years ago

mentioned in commit fca42236

Please register or sign in to reply