Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 14 10:12
    soxofaan edited #3175
  • Jan 11 17:37
    mjigmond opened #3176
  • Jan 10 15:44
    echeipesh assigned #3175
  • Jan 10 11:35
    soxofaan synchronize #3175
  • Jan 10 11:31
    soxofaan edited #3175
  • Jan 10 11:30
    soxofaan edited #3175
  • Jan 10 11:30
    soxofaan synchronize #3175
  • Jan 10 11:15
    soxofaan synchronize #3175
  • Jan 10 11:06
    soxofaan synchronize #3175
  • Jan 09 17:29
    soxofaan synchronize #3175
  • Jan 09 17:27
    soxofaan synchronize #3175
  • Jan 09 17:24
    soxofaan opened #3175
  • Jan 03 13:38
    pomadchin commented #3174
  • Jan 03 07:13
    esmeetu commented #3174
  • Jan 03 03:34
    pomadchin commented #3174
  • Jan 03 03:27
    pomadchin commented #3174
  • Jan 03 03:26
    pomadchin commented #3174
  • Jan 03 03:22
    pomadchin commented #3174
  • Jan 03 03:22
    pomadchin commented #3174
  • Jan 03 03:21
    pomadchin commented #3174
Grigory
@pomadchin
the difference is not in these tags you pointed to
it looks like the issue is that we cant read tiffs with the stripe size infeormation (in case it is a striped tiff)
Grigory
@pomadchin
Without** (typo, wrote from phone)
Simeon H.K. Fitch
@metasim
@pomadchin @echeipesh Let me know if my writupe of #3134 is unclear, and if you want more out-of-band discussion of it and what our options are around it.
Grigory
@pomadchin
@metasim :100:
Eugene Cheipesh
@echeipesh
@metasim that's super clear. Seems like there are only two options? either make abstract class Tile or push Grid and CellGrid back into trait.
Grigory
@pomadchin
the second option will definitely work
lets try the first one and see would it break smth?
Grigory
@pomadchin
Tile, ArrayTile, MultibandTile to abstract classes
ArrayMultibandTile is already a class
I’m voting for this change and it looks like it fixes everything
@echeipesh I can push some code really quick so you and @metasim can look into it
Eugene Cheipesh
@echeipesh
:+1: - lets check it out
Grigory
@pomadchin
@echeipesh @metasim locationtech/geotrellis#3136
Simeon H.K. Fitch
@metasim
:eyes: ing
I like the Tile as abstract class approach.
Grigory
@pomadchin
@metasim can you doublechek that Spark codecs would be derived?
Simeon H.K. Fitch
@metasim
Yeh, might take me a few to get everything built, etc.
Simeon H.K. Fitch
@metasim
@pomadchin I keep getting this (even after a clean):
[error] Symbol 'type geotrellis.store.hadoop.Implicits' is missing from the classpath.
[error] This symbol is required by 'package geotrellis.store.hadoop.package'.
[error] Make sure that type Implicits is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'package.class' was compiled against an incompatible version of geotrellis.store.hadoop.
(On branch pomadchin/fix/tile-hierarchy)
Eugene Cheipesh
@echeipesh
@metasim are you doing clean in away that it would get picked up by store project ?
Simeon H.K. Fitch
@metasim
:shrug:?
sbt clean
Eugene Cheipesh
@echeipesh
nope ... woops, store should be aggreagted from the root project
Simeon H.K. Fitch
@metasim
:+1:
Grigory
@pomadchin
@metasim I usually clean targets mannually rm -r ./*/target
Simeon H.K. Fitch
@metasim
gotta trust the sbt, man
Add layer to the aggregate list too
Grigory
@pomadchin
:+1:
will add it in terms of this PR
(if it would go though sparky validation check by you)
So far it's fixed those tests that were failing before (have others that are failing for other reasons)
Grigory
@pomadchin
:+1:
Grigory
@pomadchin
@metasim Im wondering what are the long term consequences of this change, hopefully none :D
Simeon H.K. Fitch
@metasim
It will limit mixins from other classes (non-traits)
I suspect nothing from a user standpoint.
Grigory
@pomadchin
@metasim does this new branch makes sense? I think we can make a binary breaking minor release with a mention not to use 3.0
Simeon H.K. Fitch
@metasim

does this new branch makes sense?

@pomadchin not sure what you're asking.

Testing out the updated PR
Grigory
@pomadchin
@metasim ah meant PR %) don’t remember what I was writing tbh, sorry for the confusion
Simeon H.K. Fitch
@metasim
I think with the tweak to ProjectedRasterLike things are OK.
However, I'm getting a different issue, this time with the gdal bindings. This is my first time using the most recent versions, so it could just be a configuration issue, but wondering if you have tips. Here's the error I'm getting:
[1 of 1000] DEBUG(1) CPLE_None(0) "No error." HTTP: libcurl/7.54.0 LibreSSL/2.0.20 zlib/1.2.11 nghttp2/1.24.0
[2 of 1000] DEBUG(1) CPLE_None(0) "No error." VSICURL: GetFileSize(https://modis-pds.s3.amazonaws.com/MCD43A4.006/24/03/2018001/MCD43A4.A2018001.h24v03.006.2018010031831_B03.TIF): response_code=0, msg=Could not resolve host: modis-pds.s3.amazonaws.com
[3 of 1000] DEBUG(1) CPLE_None(0) "No error." VSICURL: HEAD did not provide file size. Retrying with GET
[4 of 1000] DEBUG(1) CPLE_None(0) "No error." VSICURL: GetFileSize(https://modis-pds.s3.amazonaws.com/MCD43A4.006/24/03/2018001/MCD43A4.A2018001.h24v03.006.2018010031831_B03.TIF): response_code=0, msg=Could not resolve host: modis-pds.s3.amazonaws.com
[5 of 1000] DEBUG(1) CPLE_None(0) "No error." VSICURL: Request at offset 0, after end of file
[6 of 1000] FAILURE(3) CPLE_HttpResponse(11) "HTTP response." CURL error: Could not resolve host: modis-pds.s3.amazonaws.com
[7 of 1000] DEBUG(1) CPLE_None(0) "No error." VSICURL: Request at offset 0, after end of file
[8 of 1000] FAILURE(3) CPLE_OpenFailed(4) "Open failed." `/vsicurl/https://modis-pds.s3.amazonaws.com/MCD43A4.006/24/03/2018001/MCD43A4.A2018001.h24v03.006.2018010031831_B03.TIF' not recognized as a supported file format.
It's like the S3 url rewriting/conversion isn't happening
It's a bug in RF, just trying to find the change in behavior from GT 2.3 + gt-contrib
Simeon H.K. Fitch
@metasim
If nothing jumps out at you, ignore.
Yeh, I think there's something wrong with my DNS resolution. Sorry for the noise.
Grigory
@pomadchin
@metasim yoyo can you share me about how do you use this ProjectedRasterLike?
I mean why should it has a constrain inside?
btw you can use our vsicurl parser