Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Dec 13 16:25
    atararaksin edited #3164
  • Dec 13 16:24
    atararaksin synchronize #3164
  • Dec 10 16:58
    pomadchin labeled #3164
  • Dec 10 16:48
    atararaksin commented #3160
  • Dec 10 16:47
    atararaksin edited #3164
  • Dec 10 16:47
    atararaksin commented #3160
  • Dec 10 16:32
    atararaksin opened #3164
  • Dec 10 09:17
    w19930210 opened #3163
  • Dec 09 19:19
    atararaksin commented #3160
  • Dec 09 16:22

    pomadchin on master

    In GDALDataset, lazy compute of… (compare)

  • Dec 09 16:22
    pomadchin closed #3162
  • Dec 09 16:22
    pomadchin closed #3148
  • Dec 09 15:55
    pomadchin commented #3160
  • Dec 09 15:54
    pomadchin commented #3160
  • Dec 09 15:52
    pomadchin labeled #3160
  • Dec 09 15:47
    atararaksin commented #3160
  • Dec 09 14:47
    vpipkt edited #3162
  • Dec 09 14:47
    vpipkt synchronize #3162
  • Dec 09 14:46
    vpipkt edited #3162
  • Dec 09 14:46
    vpipkt edited #3162
Grigory
@pomadchin
hey @iceland1906 how do you get your MultiPolygon? from geoJson?
iceland1906
@iceland1906
yes
polygonType match {
      case "Polygon" =>
        val polygon = jsonString.parseGeoJson[Polygon]
        val reprojectedPolygon = polygon.reproject(CRS.fromEpsgCode(polygonEpsg), CRS.fromEpsgCode(epsgCode))
        MultiPolygon(Array(reprojectedPolygon))
      case "MultiPolygon" =>
        val multiPolygon = jsonString.parseGeoJson[MultiPolygon]
        multiPolygon.reproject(CRS.fromEpsgCode(polygonEpsg), CRS.fromEpsgCode(epsgCode))
    }
Grigory
@pomadchin
gotcha
and what gt version do you use?
2.x?
(Im trying with 3.x the same issue is present)
iceland1906
@iceland1906
my version 2.2.0
Grigory
@pomadchin
gotcha
Grigory
@pomadchin
@iceland1906 can you split manually your multipolygon into two polygons, parse them and check your function?
I think I know where is the problem but want to doublecheck the behavior
also will create an issue in a while
iceland1906
@iceland1906
sure
iceland1906
@iceland1906
@pomadchin it works if split the multipolygon into two seperate polygons
Simeon H.K. Fitch
@metasim
Unification with JTS geometry in GT 3.0 is awesome!
Only change I've had to do is delete any jtsGeom, except for on Extent, where I call toPolygon.
Was there consideration for making Extent an alias for Envelope?
(Mixed feelings about it myself)
Grigory
@pomadchin
@metasim there was, I don’t remember details, but AFAIK there is some sligh behavior differene between two of these // just lots of API changes
mb next release we’ll have only Envelope :shrug: mb @echeipesh remembers more
Eugene Cheipesh
@echeipesh
@metasim jts Envelope is supe heavy duty and mutable class and Extent is very heavily used throughout codebase. Basically it looked like it was introducing too much leakage and too much potential mutability.
Grigory
@pomadchin
@iceland1906 gotcha thanks! we have a corrupted json parser for multipolygons ):
will create an issue with details
iceland1906
@iceland1906
Nice! thanks for sorting this out! @pomadchin
Eugene Cheipesh
@echeipesh
@metasim that's not set in stone. If you think it would really simplify cliet code I'd be game to give it another shot for next release but basically we were in "this is not making things better for the few days we have to spend at this" category.
Grigory
@pomadchin
@iceland1906 suprsigingly it looks like it never worked
@echeipesh @metasim I’m totally fine with having both envelope and extents everywhere with extent being more restrcited but being simple and immutable; PostGIS also has two separate functions where ST_extent works with bbox and ST_envelope works with aribtrary geometries and can even work with lines / points
Simeon H.K. Fitch
@metasim
Mutability: say no more
Eugene Cheipesh
@echeipesh
Maybe we can get an immutable envelope into next JTS version? That seems ideal.
Grigory
@pomadchin
:+1: // sounds like +1000
Simeon H.K. Fitch
@metasim
@jnh5y ?
+1e50
For a few days, you guys killed it!
James Hughes
@jnh5y
For these immutable envelopes, will they be sealed already? If so, how do you expect to get the birthday card inside?;)
I like the idea; ask in the JTS channel.
Simeon H.K. Fitch
@metasim
In functional programming, the birthday card doesn't exist until you open the envelope.
James Hughes
@jnh5y
I don't know if we'll be in a situation where we'd need 'Envelope' to be an interface, etc, etc
Grigory
@pomadchin
oh no, a function that will create a copy of an envelope with a card inside
Simeon H.K. Fitch
@metasim
of course... silly me
James Hughes
@jnh5y
how does structural sharing interact with Platonic ideals?
Simeon H.K. Fitch
@metasim
There's a Platonic notion of sharing, encompassing the set of all possible relations
@pomadchin Did you guys run into this error when developing GT 3.0?
com.fasterxml.jackson.databind.JsonMappingException: Incompatible Jackson version: 2.9.8
Grigory
@pomadchin
@metasim sure
Simeon H.K. Fitch
@metasim
Oddly, it doesn't show up in the evicted report.
Simeon H.K. Fitch
@metasim
:win:
Grigory
@pomadchin
it is because of a new aws sdk :/
Simeon H.K. Fitch
@metasim
:sadness:
Grigory
@pomadchin
iceland1906
@iceland1906
:+1: