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

add the option vertexShaderCommon to prepend the custom vertexshader … #12327

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mickae1
Copy link

@mickae1 mickae1 commented Nov 22, 2024

When you want to set a custom shader, you have to include yourself all the content of the file PolylineCommon.glsl
Now with this option you can include it automatically when cesium is building the geometry.

By default the option is not activated and I've tested it with a developper tools in chrome.

It's very usefull because now I've only my code in the custom vertex shader.

Please review it, let me know if i've to change something.

Micka,

Copy link

github-actions bot commented Nov 22, 2024

Thank you for the pull request, @mickae1! Welcome to the Cesium community!

In order for us to review your PR, please complete the following steps:

Review Pull Request Guidelines to make sure your PR gets accepted quickly.

@mickae1
Copy link
Author

mickae1 commented Nov 26, 2024

@jjspace

Thank you for the pull request, @mickae1! Welcome to the Cesium community!

In order for us to review your PR, please complete the following steps:

Review Pull Request Guidelines to make sure your PR gets accepted quickly.

I've made what is needed :

sign the CLA & edit the file CONTRIBUTORS.md

@ggetz
Copy link
Contributor

ggetz commented Nov 26, 2024

Hi @mickae1, thank you for the proposal! We'll review this shortly.

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.

3 participants