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

Reading shape files #5

Open
airbreather opened this issue Apr 29, 2018 · 5 comments
Open

Reading shape files #5

airbreather opened this issue Apr 29, 2018 · 5 comments

Comments

@airbreather
Copy link
Member

From @trailway on July 19, 2016 17:57

I recently encountered some shape files which the reader does not handle. It can be debated whether the files are correct but they do work in ArcMap! What happens is that the content length does not include M content which is present (although it's presence is benign because the value says that it is not meaningful). This makes the reader fail because it walks through shape geometry using content length, i.e. it expects the next element to begin at the location just after the location where the previous element ended. This works in ArcMap because they use the shape index. I think that using the shape index (which is a required file) is a more stable approach.

Copied from original issue: NetTopologySuite/NetTopologySuite#112

@airbreather
Copy link
Member Author

From @DGuidi on July 20, 2016 6:56

can you post some sample data that actually show this failure?

@airbreather
Copy link
Member Author

From @trailway on July 22, 2016 17:22

Sample data:

Matt.zip

@airbreather
Copy link
Member Author

From @DGuidi on July 23, 2016 13:52

Thanks for sharing this :)
Hope to take a look as soon as possible.

----- Messaggio originale -----
Da: "trailway" [email protected]
Inviato: ‎22/‎07/‎2016 19:22
A: "NetTopologySuite/NetTopologySuite" [email protected]
Cc: "Diego Guidi" [email protected]; "Comment" [email protected]
Oggetto: Re: [NetTopologySuite/NetTopologySuite] Reading shape files (#112)

Sample data:
Matt.zip

You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@airbreather
Copy link
Member Author

OpenJUMP 1.13 loads this too.

@airbreather
Copy link
Member Author

This is a really old issue, and so maybe it's been fixed in the couple of years since it was reported, but it seems to work for me today:
image

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