locationtech-labs / geopyspark

GeoTrellis for PySpark
Other
179 stars 59 forks source link

Expose SpaceTimePartitionStrategy #642

Closed jdries closed 6 years ago

jdries commented 6 years ago

Currently Geopyspark supports a spatialPartitionStrategy, but this does not work for spacetime layers. Geotrellis does support this, so it would be nice if it is exposed in GeoPyspark.

jbouffard commented 6 years ago

Hello, @jdries! I agree, that would be a good feature to have in GPS. We'll try and get it for you sometime this upcoming week. Thank you for the suggestion!

Also, the SpatialPartitionerStrategy will work on spatial-temproal layers, however, they will only be partitioned by their spatial components.

jbouffard commented 6 years ago

@jdries I'm working on this issue right now and will hopefully have a PR up for it later today. Would you be interested in reviewing the Python API portion of it? It would be nice to have a user's perspective.

jdries commented 6 years ago

Certainly, I'd be happy to provide some feedback!

Jeroen Dries TAP - Centre for Remote Sensing and Earth Observation Processes VITO NV | Boeretang 200 | 2400 Mol tel. +32 14 33 55 11 | jeroen.dries@vito.behttps://outlook.vito.be/OWA/redir.aspx?C=bkbvq29Z1EO1hvVK0QJMbACOitzQ7c8Ip7CWFVkhZV75OTezDjCIkdDzkCq1rif2MTOFHY6bxtg.&URL=mailto%3avoornaam.naam%40vito.be [cid:image001.jpg@01CDCE3A.4E197A50]https://outlook.vito.be/OWA/redir.aspx?C=bkbvq29Z1EO1hvVK0QJMbACOitzQ7c8Ip7CWFVkhZV75OTezDjCIkdDzkCq1rif2MTOFHY6bxtg.&URL=http%3a%2f%2fwww.vito.be%2f[cid:image001.jpg@01CD3F0D.B57AA190]https://outlook.vito.be/OWA/redir.aspx?C=bkbvq29Z1EO1hvVK0QJMbACOitzQ7c8Ip7CWFVkhZV75OTezDjCIkdDzkCq1rif2MTOFHY6bxtg.&URL=http%3a%2f%2fwww.linkedin.com%2fcompany%2f17947[cid:image003.jpg@01CD3F0D.B57AA190]https://outlook.vito.be/OWA/redir.aspx?C=bkbvq29Z1EO1hvVK0QJMbACOitzQ7c8Ip7CWFVkhZV75OTezDjCIkdDzkCq1rif2MTOFHY6bxtg.&URL=http%3a%2f%2ftwitter.com%2f%23%21%2fvitobelgium[cid:image002.jpg@01CD3F0D.B57AA190]https://outlook.vito.be/OWA/redir.aspx?C=bkbvq29Z1EO1hvVK0QJMbACOitzQ7c8Ip7CWFVkhZV75OTezDjCIkdDzkCq1rif2MTOFHY6bxtg.&URL=https%3a%2f%2fwww.facebook.com%2fVITObelgium


Van: Jacob Bouffard notifications@github.com Verzonden: maandag 19 maart 2018 13:14 Aan: locationtech-labs/geopyspark CC: Dries Jeroen; Mention Onderwerp: Re: [locationtech-labs/geopyspark] Expose SpaceTimePartitionStrategy (#642)

@jdrieshttps://github.com/jdries I'm working on this issue right now and will hopefully have a PR up for it later today. Would you be interested in reviewing the Python API portion of it? It would be nice to have a user's perspective.

- You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/locationtech-labs/geopyspark/issues/642#issuecomment-374191883, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFqXyLC61OkW8tXzQMHG0eX23AfUryiVks5tf6EtgaJpZM4StfQ0.

Indien u VITO Mol bezoekt, hou aub er dan rekening mee dat de hoofdingang voortaan enkel bereikbaar is vanuit de richting Dessel-Retie, niet vanuit richting Mol, zie vito.be/route.http://www.vito.be/route If you plan to visit VITO at Mol, then please note that the main entrance can only be reached coming from Dessel-Retie and no longer coming from Mol, see vito.be/en/contact/locations.http://www.vito.be/en/contact/locations VITO Disclaimer: http://www.vito.be/e-maildisclaimer

jbouffard commented 6 years ago

Great! Thank you! I'll let you know when I have the PR up

jbouffard commented 6 years ago

@jdries The PR is up! #643