-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Labels
!OptimizationThe feature is to improve performance.The feature is to improve performance.StaleThis issue is stale, no activity for 90 days. Remove stale label or comment within 30 days.This issue is stale, no activity for 90 days. Remove stale label or comment within 30 days.X Re-EvaluateDeprecated (but keep) - Look more into reviving this issue. Used for migration from 1.x to 2.xDeprecated (but keep) - Look more into reviving this issue. Used for migration from 1.x to 2.x
Milestone
Description
Right now we're using a custom !UnifiedGridCoverage class to stitch together multiple TIF files for our elevation data. We should probably really be using an !ImageMosaic. The main reason we're not already is because the naive approach we're doing is roughly 200x faster than using the !ImageMosaic. I've emailed back and forth with Simone Giannecchini of the !GeoTools project who thinks we can optimize this:
I am pretty sure we can optimize this 'elevation service' even more.
As an instance you should supply a !GridGeometry2D to the reader and
also you should create it with the IGNORE OVERVIEWS hint in order to
get just a small area around the point you need.
Not a huge priority, but something it'd be nice to get to some day.
Metadata
Metadata
Assignees
Labels
!OptimizationThe feature is to improve performance.The feature is to improve performance.StaleThis issue is stale, no activity for 90 days. Remove stale label or comment within 30 days.This issue is stale, no activity for 90 days. Remove stale label or comment within 30 days.X Re-EvaluateDeprecated (but keep) - Look more into reviving this issue. Used for migration from 1.x to 2.xDeprecated (but keep) - Look more into reviving this issue. Used for migration from 1.x to 2.x