-
Notifications
You must be signed in to change notification settings - Fork 123
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
Hillshading accent color #725
Comments
In the current code - didn't find also where the highlight and shadow colors colors are gathered and applied. It should probably be in For the accent color implementation the example code from maplibre is extracted below :
|
The current implementation uses the same technique as the https://openlayers.org/en/latest/examples/shaded-relief.html example. No colors can be configured. To make the result look good, a version of OpenLayers that includes openlayers/openlayers#14282 is needed. That was a big change and makes a big difference. If someone can provide a pull request that implements the color options, it would be greatly appreciated. |
Implementation hints: The rendering transformation takes place here: ol-mapbox-style/src/shaders.js Lines 1 to 119 in f5c134c
The Lines 819 to 846 in f5c134c
This is where the |
Merci beaucoup Virgil ! If you have some additional bandwidth we are also looking at the line-pattern implementation #713 |
Should be ready to be published in a stable release version |
@ahocevar We'd like to use the functionality on our website, any chance you could include it in a new release version of the plugin soon ? |
@utagawal Please be patient. I have a lot of work that pays my bills to finish before I can maintain this and other libraries in my free time. Speaking of, I'm grateful for your one-time sponsoring a while ago. I used those funds for adding the first iteration of the hillshading capability, which was quite an involved change that also required changes to the OpenLayers core library. |
@utagawal I published v9.3.0. |
Many thanks @ahocevar |
For the accent color implementation in hillshading , maybe usefull to take inspiration from here : src/shaders/hillshade.fragment.glsl
from https://github.com/mapbox/mapbox-gl-js/pull/5286/files#diff-a826d96571ff241535a6bac52167defaabc13c91883e2529cf84608c70ca9b79
(but it's using webGL)
The text was updated successfully, but these errors were encountered: