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.

Sign up for GitHub

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

Jump to bottom

GeometryFixer changes GeometryType from MultiPolygon to Polygon #566

Closed
swidz opened this issue Oct 25, 2021 · 1 comment
Closed

GeometryFixer changes GeometryType from MultiPolygon to Polygon #566

swidz opened this issue Oct 25, 2021 · 1 comment
Labels
upstream-reported The issue appears to be present in JTS as well, and it has been reported there.
Milestone
2.5

Comments

@swidz
Copy link

swidz commented Oct 25, 2021 •
edited

In some cases, when NetTopologySuite.Geometries.Utilities.GeometryFixer.Fix(geometry) is applied to MultiPolygon geometry, Polygon geometry type is returned.
Is this expected?

Please see the included log for geometry details
PolygonVsMultiPolygonTest-637707715546023206.zip

Regards,
Sebastian
FObermaier added a commit that referenced this issue Oct 26, 2021
@FObermaier
Add unit tests for #566 and 567
d793cd2
@FObermaier
Copy link
Member

FObermaier commented Oct 26, 2021 •
edited

That is the way it is intentionally coded. When working with data stores that require not to change the typ I see a reason to avoid that. We'll check with JTS.

@FObermaier FObermaier mentioned this issue Oct 26, 2021
Closed
@FObermaier FObermaier added the upstream-reported The issue appears to be present in JTS as well, and it has been reported there. label Oct 26, 2021
@swidz swidz mentioned this issue Oct 26, 2021
Closed
@FObermaier FObermaier added this to the 2.5 milestone Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
upstream-reported The issue appears to be present in JTS as well, and it has been reported there.
Projects
None yet
Milestone
2.5
Development

No branches or pull requests
2 participants
@FObermaier @swidz