-
Notifications
You must be signed in to change notification settings - Fork 64
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
Fix logic checking periodic model faces #446
Conversation
/runtests |
Test Result: success (details) |
/runtests |
@bobpaw I'm running tests again (which now includes CGNS tests). If those pass I have no objections to merging this. |
Build Log |
@joshia5 Would you mind rebasing this on develop? The CGNS tests are failing as this branch is missing some fixes. |
1966945
to
2e031e0
Compare
/runtests |
Build Log |
Signed-off-by: Aiden Woodruff <[email protected]>
Suppress warning by casting to void. Signed-off-by: Aiden Woodruff <[email protected]>
Suppress warning by casting to void. Signed-off-by: Aiden Woodruff <[email protected]>
Suppress warning by casting to void. Signed-off-by: Aiden Woodruff <[email protected]>
Fix bug in logic checking periodic model faces when computing parametric coordinates
This PR is relating to issue #444
The PR is correcting the logic that checks if case is periodic and then only execute the lines
core/ma/maSnap.cc
Lines 502 to 511 in 60c507f