-
Notifications
You must be signed in to change notification settings - Fork 362
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Investigate WMS for region based csv support #244
Comments
I created a raster of the post codes and have hooked up a request tile extension to the WMSTileImageryProvider to color the image based on the pixel ID. So far the performance is good and it scales well. |
Getting this to work in leaflet will be another matter since it requires fiddling with the tiles. |
Kevin, I'm trying to find the best way to return an id in the color value of a pixel. Anything from 0-255 is relatively simple and beyond that looks rather custom (return bil, etc). This is very close to the WMS GeoServer Terrain problem. Did you ever spend any time with that? |
I did spend some time with that, and had a lot of trouble. Geoserver (at least at the time) was very much oriented around 8 bits per channel. So even if the source data (coming from a TIF in my case) had more precision, but the time an output plugin saw it, it had already been downsampled. The BIL plugin had a hacky way around that. That's why our approach was to write a simple proxy that accepted a full-precision TIF file and converted it to an RGB-encoded PNG on the fly. But to be fair, I was doing this last minute for some demo and didn't have the time to really figure it out. There could be a better solution I missed. |
I did see this in the issues chain which points toward a pre-build the tile tree solution. We can talk more when we meet today. |
Sounds good. I'm struggling to remember all the details (it was almost exactly two years ago that I was working on this), but I'll tell you what I know. |
I'm closing this one since the investigation part is over and the implementation part moves to issue #237. |
No description provided.
The text was updated successfully, but these errors were encountered: