Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [sfcurve-dev] Addition of GeoWave Indexing Code To SFCurve
Hi James,

Thanks for the PR.  I commented on it to note a number of the details we'll have to work out before we can merge the work.

I think we will need to raise a number of Committer Questionnaires (CQs) for the PR.  Can you identify the dependencies that this work pulls in?

Cheers,

Jim

On 06/13/2017 11:34 AM, James McClain wrote:
Hello,

I made the following pull request: https://github.com/locationtech/sfcurve/pull/18 in which I have imported the GeoWave indexing code into the SFCurve project.

The is a direct importation with no changes, but the code does compile and can be published locally (sbt "project geowave-index" publishLocal) and tests do pass (sbt "project geowave-index" test).  Small-scale tests with GeoTrellis show that that the code appears to function.

I am writing to ask for comments and/or start a conversation about the pull request mentioned above.

Thank you,
James McClain

--
James McClain
Software Developer
Azavea  |  990 Spring Garden Street, 5th Floor, Philadelphia, PA 19123



_______________________________________________
sfcurve-dev mailing list
sfcurve-dev@xxxxxxxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.locationtech.org/mailman/listinfo/sfcurve-dev


Back to the top