-
Notifications
You must be signed in to change notification settings - Fork 824
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
Enable transparencyAsCoverage in Babylon Viewer #4531
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Thank you! Is it working? You may want to update the babylon.js version while you're at it; it's probably quite out of date. |
No idea. I am not having luck getting it to run locally. I was hoping the CI can run it instead. |
EDIT: Never mind. It works. I just need to rescan. |
I don't know what
Maybe as a separate PR to limit the changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
So, our CI does't actually run any renderers besides |
Hmm, I don't see any information on this, so I have no idea what you are talking about. |
Ok, just tell me what I need to do. I can't even get the |
Okay, if your dev environment isn't working then I'll assume you know how to write Babylon code like this blindly. I'll go ahead and merge it. If @bhouston has any trouble making babylon goldens, he can ping you here or there. |
I would definitely prefer for it to work, but let me know if something goes wrong with this change. |
Reference Issue
See KhronosGroup/glTF-Sample-Assets#34 (comment)