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

Allow unstructured grids with more than typemax(Int32) points #106

Merged
merged 3 commits into from
Jul 1, 2022

Conversation

jipolanco
Copy link
Member

Fixes #105

[skip ci]
@PetrKryslUCSD
Copy link

Seems to be working! Many thanks!

@jipolanco
Copy link
Member Author

Great, thanks for testing!

@jipolanco jipolanco merged commit db26ecc into master Jul 1, 2022
@jipolanco jipolanco deleted the unstructured-int branch July 1, 2022 14:53
@henry2004y
Copy link

Today I noticed this change due to a test failure in my package that depends on WriteVTK.jl. The differences are simply caused by forcing connectivity and offsets to be Int64 instead of Int32. I guess this will slightly increase the file sizes? Is it possible to support both depending on the input connectivity type?

@jipolanco
Copy link
Member Author

Is it possible to support both depending on the input connectivity type?

Yes, that would make sense. I can't look at this right now, but I should be able to add this within the next couple of weeks.

@jipolanco
Copy link
Member Author

@henry2004y this should be now fixed by #107 (v1.14.4).

@henry2004y
Copy link

Great! Thank you!

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.

Inexact Error: ran out of numbers
3 participants