Skip to content

Example t8_example_geometries.cxx is not working as expected #1517

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

Closed
lenaploetzke opened this issue Mar 27, 2025 · 3 comments · Fixed by #1582
Closed

Example t8_example_geometries.cxx is not working as expected #1517

lenaploetzke opened this issue Mar 27, 2025 · 3 comments · Fixed by #1582
Assignees
Labels
priority:medium Should be solved within half a year workload:low Would take half a day or less

Comments

@lenaploetzke
Copy link
Collaborator

Bug report

Describe the bug
When I run the geometry example, e.g., for geometry number 0 (T8_GEOM_SINCOS), i get the error

[t8] Creating uniform level 2 forest with a sinus/cosinus geometry.
[t8 0] Constructed the geometry_handler.
[t8 0] Registering geometry with name t8_sincos_geometry
[t8 0] Cmesh is not partitioned.
[t8 0] committed cmesh with 2 local and 2 global trees and 0 ghosts.
[t8 0] t8_geometry_sincos is not compatible with tree type Hex
 It is only compatible with quad elements.
[t8 0] Detected incompatible geometry for tree 0

This surprises me in particular because the tree class is set to Quad in lines 695f.
Comparable errors are thrown when I call the example with other geometries.

To Reproduce
For example with
./t8_example_geometries -l 2 -g 0

Estimated priority
"Priority: medium" Should be solved within half a year

@holke
Copy link
Collaborator

holke commented Mar 27, 2025

@sandro-elsweijer
i have a feeling this might be related to when we dropped the dimension parameter from the geometry and made geometries valid for all dimensions.

@sandro-elsweijer
Copy link
Collaborator

@holke Yes its from that pr but that is not the reason, since the dimension member variable was never checked or used after being set

@sandro-elsweijer
Copy link
Collaborator

I can take a look at this after my holiday :)

@Davknapp Davknapp added priority:medium Should be solved within half a year workload:low Would take half a day or less labels Apr 14, 2025
@t8ddy t8ddy moved this from In-Box to ToDo in t8code's Issue Landing page Apr 14, 2025
@t8ddy t8ddy moved this from ToDo to In Progress in t8code's Issue Landing page Apr 14, 2025
This was referenced Apr 14, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in t8code's Issue Landing page Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:medium Should be solved within half a year workload:low Would take half a day or less
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants