Simeon H.K. Fitch comments

Results 210 comments of


                                            Simeon H.K. Fitch

EMR zeppelin `%sql` error with TIleUDT

@vpipkt Any clue if this is still an issue? Not quite sure how this use case is any different from other standard use cases.

Determine how `stitch` or the use of it handles time.

Confirmed: Temporal component is discarded... https://github.com/s22s/raster-frames/blob/4434e36b825374d93c3002091df0b2c34bd045b1/src/main/scala/astraea/spark/rasterframes/RasterFrameMethods.scala#L150-L154 ... with merges according to however `tileToLayout` wants to do it: https://github.com/s22s/raster-frames/blob/4434e36b825374d93c3002091df0b2c34bd045b1/src/main/scala/astraea/spark/rasterframes/RasterFrameMethods.scala#L162

Determine how `stitch` or the use of it handles time.

To close this out, add ScalaDoc to explicitly state this behavior.

Support both integer and double forms of aggregate histograms

Should just require creating an `Int` version of this: https://github.com/s22s/raster-frames/blob/5fb23a260d514478c5480f847572b90104326b30/src/main/scala/astraea/spark/rasterframes/functions/AggregateHistogramFunction.scala#L32

`toRaster` is slow even for very small images

``` "Stack Trace";"Sample Count";"Percentage(%)" "geotrellis.raster.merge.SinglebandTileMergeMethods$class.merge(SinglebandTileMergeMethods, Extent, Extent, Tile, ResampleMethod)";"1,277";"61.721" " geotrellis.raster.package$withTileMethods.merge(Extent, Extent, Tile, ResampleMethod)";"1,277";"61.721" " geotrellis.raster.package$withTileMethods.merge(Extent, Extent, Object, ResampleMethod)";"1,277";"61.721" " geotrellis.spark.tiling.CutTiles$$anonfun$apply$1$$anonfun$apply$2.apply(Tuple2)";"1,277";"61.721" " geotrellis.spark.tiling.CutTiles$$anonfun$apply$1$$anonfun$apply$2.apply(Object)";"1,277";"61.721" ```

Feature request: Provide runtime function to get a bindings identifier

Could probably be addressed with the maven equivalent of `sbt-buildinfo`

`FromSql` is not implemented for `Vec<f64>`

> I think you can decode it as an `Value::List`: https://docs.rs/duckdb/latest/duckdb/types/enum.Value.html#variant.List Is there a way of doing this without having to iterate over `Vec` and convert each element into `f64`,...

Question: Double-Ended IndexMap

I have a similar use case, and ended up here.

OpenTelemetry Integration

Since this crate already uses `tracing`, and the user can select [OpenTelemetry][1] as the subscriber, wouldn't additional instrumentation of calls be sufficient integration to support OTEL? Or is it more...

Question about Send + Sync

I have the same issue.