-
Notifications
You must be signed in to change notification settings - Fork 119
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
add test_mesh_corners3.c #235
Conversation
Thanks! Can I ask you to add a file doc/author_aiton.txt like the others to release your code under BSD? We'll take care updating the Makefiles then. |
6b46693
to
533368b
Compare
We discuss potential changes to the |
Thanks, if I may just ask to format the test program according to our customs (e. g. indentation, no declarations after code within a block) and slightly beyond (liberally adding comments every couple lines explaining what's going on). |
Yes, I'll work on this. |
Yes, I'll work on this.
During my work on PR #262 I also implemented an inter-tree test case that checks the implementation of edgehanging corner neighbors across tree faces and edges. I would propose to add this to the PR as well.
Sounds perfect!
|
I've added the inter-tree test cases and adapted indentation, comment format and the positions of variable declarations to fit the standard p4est format. |
Thanks! Sorry for being obnoxious, but I'd caution against using TRUE variables in general. It is too tempting to compare == TRUE, which would be wrong. I'm fine with assigning = FALSE or = 0 or = !FALSE or = 1, which is legal. |
Nice thanks so much! |
Here's the test program that @hannesbrandt requested, which demonstrates the missing corner information I'm running into with p8est_mesh.