Skip to content

Conversation

@capitalaslash
Copy link
Contributor

No description provided.

@nschloe
Copy link
Owner

nschloe commented Oct 24, 2021

That fails, but it's a meshio issue. It should probably warn that it can't write cell_sets yet, and that one should convert it to int_data first.

@capitalaslash
Copy link
Contributor Author

I understand that the write out is performed in meshio, but the fact that pygmsh is not capable of printing out physical (and geometrical) tags, something that is required for any gmsh file, is a serious limitation in my opinion.
#423 is a solution that does not require to touch meshio. I tried some months ago to get this to work by modifying meshio, but I gave up due to being far more involved than going the #423 way.

@nschloe
Copy link
Owner

nschloe commented Oct 24, 2021

Is sets_to_int_data not good enough then?

@capitalaslash
Copy link
Contributor Author

I tried that but it fails. Turns out the cell_sets are wrong: that is another problem with meshio.
I am looking into it.
If I use gmsh.write() the output is ok.

@nschloe
Copy link
Owner

nschloe commented Oct 24, 2021

Turns out the cell_sets are wrong: that is another problem with meshio. I am looking into it.

If I understand correctly, fixing this will fix your problem.

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

Successfully merging this pull request may close these issues.

2 participants