-
Notifications
You must be signed in to change notification settings - Fork 90
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
Minor FCI improvements #2201
Minor FCI improvements #2201
Conversation
Because we construct the FCI transform _during_ the construction of Coordinates, we can't call `Mesh::getCoordinates` in the `FCIMap` constructor. An alternative workaround would be to call `Mesh::get(dy)` instead, but the implemented method is more explicit
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.
clang-tidy made some suggestions
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.
clang-tidy made some suggestions
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.
Looks good to me.
i/k_corner
andt_x/z
stuff is no longer used so can just goBOUT_FOR
BOUT_FOR_SERIAL
as it's got avector.push_back
in the loop and I suspect that's not thread safedy
to avoid null-dereferenceCoordinates
constructor, which means we can't callMesh::getCoordinates
. Instead, pass indy
(It's a bit easier to view the diff if you select "hide whitespace changes")