Skip to content
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

new leaflet sample [work in progress] #4

Merged
merged 6 commits into from
Jan 22, 2016
Merged

Conversation

jgravois
Copy link

work in progress (toward #3)
screenshot 2016-01-22 09 48 13

  • do something to account for 257x257px tiles?
  • standardize color ramp displayed for individual tiles
  • get rid of white band at international dateline
  • figure out how to dig into the actual elevation value to display a value on mousemove
  • repackage code as a custom Leaflet layer?

@jgravois
Copy link
Author

@lucianpls i just layed down another commit so the sample now digs elevation values out of the data directly. (its based on the tutorial here).

the next step is to define a static range/colormap for all tiles...

@jgravois
Copy link
Author

getting closer...

screenshot 2016-01-22 12 08 15

@lucianpls
Copy link
Contributor

What you display now is the value of the pixel, not the elevation. You can't interrogate the canvas for that, you have to get the corresponding value from the decoded LERC array.

@lucianpls
Copy link
Contributor

@jgravois
There will be cases where the LERC layer will be an overlay on top of a basemap. Then the NoData areas should be transparent so that the basemap will be seen.

@lucianpls
Copy link
Contributor

@jgravois
The white edge (left and top too) is a known issue, in the tiles themselves. Those pixels are flagged by the mask as NoData.

@jgravois
Copy link
Author

alright, i pushed a few more commits, cleaned up the white border and made min/max values configurable via user input.

if anyone can help with figuring out how to dig out the elevation data every time the mouse is moved, that would be really sweet.

tmaurer3 added a commit that referenced this pull request Jan 22, 2016
new leaflet sample [work in progress]
@tmaurer3 tmaurer3 merged commit 9c7ac72 into Esri:master Jan 22, 2016
@patrickarlt
Copy link
Contributor

@lucianpls now that we have #5 done how would we convert a x/y pixel position on the tile to a value from the decoded array?

@lucianpls
Copy link
Contributor

I got it working, but sometimes it says "undefined", not sure why.

@jgravois jgravois deleted the leaflet branch February 1, 2016 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants