forked from bendudson/hermes-2
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Performance improvements #7
Open
dschwoerer
wants to merge
10
commits into
fci-bs
Choose a base branch
from
perf
base: fci-bs
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commits on Nov 15, 2022
-
In case it is accidentially used on BoutReals do not convert them to fields
Configuration menu - View commit details
-
Copy full SHA for 06d27bc - Browse repository at this point
Copy the full SHA 06d27bcView commit details -
Add floor_all version for pressure
Directly applying on the pressure avoids the need to recalculate Pe everywhere
Configuration menu - View commit details
-
Copy full SHA for 0875931 - Browse repository at this point
Copy the full SHA 0875931View commit details -
Configuration menu - View commit details
-
Copy full SHA for ea166c3 - Browse repository at this point
Copy the full SHA ea166c3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 82af949 - Browse repository at this point
Copy the full SHA 82af949View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0082e29 - Browse repository at this point
Copy the full SHA 0082e29View commit details
Commits on Nov 17, 2022
-
Configuration menu - View commit details
-
Copy full SHA for 46742f0 - Browse repository at this point
Copy the full SHA 46742f0View commit details -
This has a higher overhead in terms of computing, but we need to move less data, which is a net plus
Configuration menu - View commit details
-
Copy full SHA for a92d1f6 - Browse repository at this point
Copy the full SHA a92d1f6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 817c6d9 - Browse repository at this point
Copy the full SHA 817c6d9View commit details -
Configuration menu - View commit details
-
Copy full SHA for ec6395b - Browse repository at this point
Copy the full SHA ec6395bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5e31428 - Browse repository at this point
Copy the full SHA 5e31428View commit details
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.