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

minimal test case to check merged surface #27

Open
shimwell opened this issue Apr 19, 2022 · 1 comment
Open

minimal test case to check merged surface #27

shimwell opened this issue Apr 19, 2022 · 1 comment

Comments

@shimwell
Copy link
Member

When two surfaces are the same in the geometry then the brep file allows them to be merged into one surface.

This method currently loops through each volume and gets the surfaces for the volume, the surface are then saved as an stl file and imported again to make the h5m file

This leads to the same surface being defined twice in the geometry.

The touching surface would appear in two stl files and then it would appear in two meshes when making the pymoab mesh object

the touching surface is not overlapping but is in the same location as another surface and therefore can result in problems during particle transport.

Perhaps a minimal test case can be set up that makes this geometry and checks the conversion results in 2 volumes each with 3 regular surfaces and 1 shared surface

one volume is drawn in black lines while the other volume is drawn in red lines

Screenshot from 2022-04-19 15-13-26
.

@shimwell
Copy link
Member Author

perhaps this issue should live in the vertices to h5m package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant