Skip to content
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

Merged
merged 10 commits into from
Dec 15, 2023
Merged

Conversation

scottaiton
Copy link
Contributor

Here's the test program that @hannesbrandt requested, which demonstrates the missing corner information I'm running into with p8est_mesh.

@cburstedde
Copy link
Owner

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.

@hannesbrandt hannesbrandt self-assigned this Aug 31, 2023
@hannesbrandt
Copy link
Collaborator

We discuss potential changes to the p8est_mesh_t regarding the edge-hanging corner neighbors in #244.

@cburstedde
Copy link
Owner

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).

@hannesbrandt
Copy link
Collaborator

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.

@cburstedde
Copy link
Owner

cburstedde commented Dec 14, 2023 via email

@hannesbrandt
Copy link
Collaborator

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.

@cburstedde
Copy link
Owner

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.

@cburstedde
Copy link
Owner

Nice thanks so much!

@cburstedde cburstedde merged commit 4af6054 into cburstedde:develop Dec 15, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants