[GitHub] [incubator-sedona] jiayuasu commented on a change in pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
jiayuasu commented on a change in pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484#discussion_r500019542 ## File path: core/pom.xml ## @@ -32,10 +32,16 @@ jar + +org.locationtech.jts +jts-core +

[GitHub] [incubator-sedona] jiayuasu commented on a change in pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
jiayuasu commented on a change in pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484#discussion_r500018075 ## File path: core/pom.xml ## @@ -35,7 +35,7 @@ org.wololo jts2geojson -0.10.0 +

Re: Podling Sedona Report Reminder - October 2020

2020-10-05 Thread Jia Yu
Yes, indeed. I am not sure why. I copied the content back and now it has all sections and text. On Mon, Oct 5, 2020 at 8:39 PM Felix Cheung wrote: > Thanks- it looks like the report is truncated? It seems to be missing some > text and section > > > On Sun, Oct 4, 2020 at 11:53 PM Jia Yu wrote:

Re: Podling Sedona Report Reminder - October 2020

2020-10-05 Thread Felix Cheung
Thanks- it looks like the report is truncated? It seems to be missing some text and section On Sun, Oct 4, 2020 at 11:53 PM Jia Yu wrote: > Hi folks, > > I have submitted the draft of the report. Please take a look. Mentors, > thank you very much for your help! > > Jia > > On Sat, Oct 3, 2020

[GitHub] [incubator-sedona] Imbruced commented on pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
Imbruced commented on pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484#issuecomment-703504794 @jiayuasu I will take care of this today evening. This is an automated message from the Apache Git

[GitHub] [incubator-sedona] jiayuasu edited a comment on pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
jiayuasu edited a comment on pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484#issuecomment-703437703 @Imbruced Apparently, the Python test has failed (https://travis-ci.org/github/apache/incubator-sedona/builds/732868873) because of the path issue. But I believe

[GitHub] [incubator-sedona] jiayuasu commented on pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
jiayuasu commented on pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484#issuecomment-703437703 @Imbruced Apparently, the Python test has failed because of the path issue. But I believe it should be a quick fix. Can you fix it and push to the Sedona "jtsplus"

Re: Podling Sedona Report Reminder - October 2020

2020-10-05 Thread Jia Yu
Hi folks, I have submitted the draft of the report. Please take a look. Mentors, thank you very much for your help! Jia On Sat, Oct 3, 2020 at 5:31 PM Jia Yu wrote: > Hi Felix, > > I will write the report tomorrow. I will let you guys know once I am done. > > Thanks, > Jia > > On Sat, Oct 3,

[GitHub] [incubator-sedona] jiayuasu commented on pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
jiayuasu commented on pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484#issuecomment-703430710 @Imbruced Hi Pawel, although I think there are no actual technical changes to Sedona itself, it would be good if you can take a look at the Python part. Most changes

[GitHub] [incubator-sedona] jiayuasu opened a new pull request #484: [SEDONA-1] Make Sedona depend on a LocationTech JTS 1.16.1 fork

2020-10-05 Thread GitBox
jiayuasu opened a new pull request #484: URL: https://github.com/apache/incubator-sedona/pull/484 ## Is this PR related to a proposed Issue? #482 #435 #233 ## What changes were proposed in this PR? Please see my proposal: