You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2021/02/14 23:09:59 UTC

[GitHub] [incubator-sedona] jiayuasu opened a new pull request #509: [SEDONA-16] Use a GeoTools Maven Central wrapper to fix notebooks in Python Binder

jiayuasu opened a new pull request #509:
URL: https://github.com/apache/incubator-sedona/pull/509


   ## Is this PR related to a proposed Issue?
   
   https://issues.apache.org/jira/browse/SEDONA-16
   
   ## What changes were proposed in this PR?
   
   1. Use org.datasyslab:geotools-wrapper:geotools-24.0 in Jupyter Notebook
   2. Move Jupyter Notebook to a binder folder to enable interactive live notebooks on binder
   
   ## How was this patch tested?
   
   Tested in notebooks
   
   ## Did this PR include necessary documentation updates?
   
   Some are updated. Others to be fixed in a separate doc commit.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-sedona] jiayuasu commented on pull request #509: [SEDONA-16] Use a GeoTools Maven Central wrapper to fix notebooks in Python Binder

Posted by GitBox <gi...@apache.org>.
jiayuasu commented on pull request #509:
URL: https://github.com/apache/incubator-sedona/pull/509#issuecomment-778858495


   @Imbruced @netanel246 
   
   Folks, as I tested in Jupyter notebook, if the users want to use GeoTools lib for ShapefileReader and CRS transformation, it will definitely fail because GeoTools's jai-core issue, unless the user compiles the source code.
   
   Therefore, I moved GeoTools to Maven Central using a wrapper: https://github.com/jiayuasu/geotools-wrapper


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-sedona] jiayuasu merged pull request #509: [SEDONA-16] Use a GeoTools Maven Central wrapper to fix notebooks in Python Binder

Posted by GitBox <gi...@apache.org>.
jiayuasu merged pull request #509:
URL: https://github.com/apache/incubator-sedona/pull/509


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org