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

Making multiline strings to address international date line #5

Closed
smit1678 opened this issue Jun 1, 2012 · 6 comments
Closed

Making multiline strings to address international date line #5

smit1678 opened this issue Jun 1, 2012 · 6 comments

Comments

@smit1678
Copy link

smit1678 commented Jun 1, 2012

@springmeyer

I want to ping you on the international date line issue and see if there is a possible solution by using MultiLineStrings. Is it possible to create multiline strings when an arc crosses the date line?

Another issue with breaking into two lines is the gap that is formed between the two points that cross the date line. Is it also possible to create two points, along the arc, that would be just offset from the date line? So when the line is broken, create two points, at -179.99 and +179.99 at the appropriate latitude?

Example of a line: https://gist.github.com/39ab121a3e225c516885

@springmeyer
Copy link
Owner

ya, one sec :)

@springmeyer
Copy link
Owner

@smit1678 & @tristen - give that a shot ^^.

@springmeyer
Copy link
Owner

see #4 (comment) for some examples

@springmeyer
Copy link
Owner

ping @smit1678 - did you ever have a chance to try this ^^?

@smit1678
Copy link
Author

@springmeyer yes, sorry I didn't respond here. This work flawlessly for about 5,000 lines. I need to do a bit more testing to determine exactly but it seems there might be some bugs when getting over that. I saw a few edge cases where they were not being split into multi linestrings. I will post some more details here and keep this ticket open.

@springmeyer
Copy link
Owner

@smit1678 - okay, thanks for the update. I've not really considered how this code would perform for lots of lines, so we may need to test that more. I'm going to close this issue. Please re-open another on perf/memory usage when you get there.

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

2 participants