locationtech / jts

The JTS Topology Suite is a Java library for creating and manipulating vector geometry.
Other
1.98k stars 441 forks source link

Feature suggestion: RadialDistanceSimplifier #69

Open HarelM opened 7 years ago

HarelM commented 7 years ago

Hi, I posted the same enhancement request here: https://github.com/NetTopologySuite/NetTopologySuite/issues/142 But I was told I should post it here or in the mailing list. In short, a very simple algorithm to simplify a line according to distance between points. .Net code can be found there, I'm not a very experienced Java developer so I don't really think I should do a pull request to this repository, having said that, It would be nice to get this :-)

dr-jts commented 7 years ago

A nice idea. It would take a bit of work to make this a production-quality operation in JTS, so unless funding is available to do this it will have to wait for time to be available .

On Tue, Dec 13, 2016 at 6:19 AM, Harel M notifications@github.com wrote:

Hi, I posted the same enhancement request here: NetTopologySuite/NetTopologySuite#142 https://github.com/NetTopologySuite/NetTopologySuite/issues/142 But I was told I should post it here or in the mailing list. In short, a very simple algorithm to simplify a line according to distance between points. .Net code can be found there, I'm not a very experienced Java developer so I don't really think I should do a pull request to this repository, having said that, It would be nice to get this :-)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/locationtech/jts/issues/69, or mute the thread https://github.com/notifications/unsubscribe-auth/ADXZXUc83MFYMvp16IWtmciXnYlnvbngks5rHqlwgaJpZM4LLx_T .

dr-jts commented 7 years ago

You could also open an issue on the JTS project for this, to help in tracking this.

On Tue, Dec 13, 2016 at 6:19 AM, Harel M notifications@github.com wrote:

Hi, I posted the same enhancement request here: NetTopologySuite/NetTopologySuite#142 https://github.com/NetTopologySuite/NetTopologySuite/issues/142 But I was told I should post it here or in the mailing list. In short, a very simple algorithm to simplify a line according to distance between points. .Net code can be found there, I'm not a very experienced Java developer so I don't really think I should do a pull request to this repository, having said that, It would be nice to get this :-)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/locationtech/jts/issues/69, or mute the thread https://github.com/notifications/unsubscribe-auth/ADXZXUc83MFYMvp16IWtmciXnYlnvbngks5rHqlwgaJpZM4LLx_T .

HarelM commented 7 years ago

I'm sorry, I'm new to this fine project, can you elaborate a bit? In term of funding, I guess I can't help a lot, I'm writing my own open-source-probably-not-gonna-earn-me-a-dime project... Also I'm not sure how changes here are converted to the .Net project which is what I actually need :-/...

dr-jts commented 7 years ago

Make a GitHub issue under the JTS project in the same way you did for NTS. (No need to copy the code, though!)

On Tue, Dec 13, 2016 at 1:14 PM, Harel M notifications@github.com wrote:

I'm sorry, I'm new to this fine project, can you elaborate a bit? In term of funding, I guess I can't help a lot, I'm writing my own open-source-probably-not-gonna-earn-me-a-dime project... Also I'm not sure how changes here are converted to the .Net project which is what I actually need :-/...

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/locationtech/jts/issues/69#issuecomment-266863892, or mute the thread https://github.com/notifications/unsubscribe-auth/ADXZXcAMssfS0tZs3OF6mUOqq8f-hiHuks5rHwqkgaJpZM4LLx_T .

dr-jts commented 7 years ago

Once the code makes it way into JTS, it's up to the NTS project to port it.

On Tue, Dec 13, 2016 at 4:48 PM, Martin Davis mtnclimb@gmail.com wrote:

Make a GitHub issue under the JTS project in the same way you did for NTS. (No need to copy the code, though!)

On Tue, Dec 13, 2016 at 1:14 PM, Harel M notifications@github.com wrote:

I'm sorry, I'm new to this fine project, can you elaborate a bit? In term of funding, I guess I can't help a lot, I'm writing my own open-source-probably-not-gonna-earn-me-a-dime project... Also I'm not sure how changes here are converted to the .Net project which is what I actually need :-/...

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/locationtech/jts/issues/69#issuecomment-266863892, or mute the thread https://github.com/notifications/unsubscribe-auth/ADXZXcAMssfS0tZs3OF6mUOqq8f-hiHuks5rHwqkgaJpZM4LLx_T .

HarelM commented 7 years ago

Isn't this the GitHub project for JTS? Isn't this an issue without the code? I'm not sure I'm following... :-/

dr-jts commented 7 years ago

Whoops, yes, sorry, this is the right place for this.