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

Certain GPU drivers do not handle shader branching correctly #69

Closed
brianchirls opened this issue Aug 8, 2014 · 3 comments
Closed

Certain GPU drivers do not handle shader branching correctly #69

brianchirls opened this issue Aug 8, 2014 · 3 comments
Assignees
Labels

Comments

@brianchirls
Copy link
Owner

Originally reported here: processing/p5.js-video#40

This is tricky to replicate, as it seems to be a problem only for certain devices. It first manifested as complete lack of any alpha channel on images processed by the Chroma effect. Testing with a split effect showed that the same users were seeing white dots along the edge where the shader had branching. A version of the shader without branching solved the problem.

This ticket is to remove branching from effects wherever possible, starting with Split and Chroma.

@brianchirls brianchirls self-assigned this Aug 8, 2014
brianchirls added a commit that referenced this issue Aug 11, 2014
Hoping this will help address #69. If affected users can run the tests,
maybe the fine-grained results can provide more information about what's not working
@brianchirls
Copy link
Owner Author

@Xeronimo74 Would you care to take the examples for a spin and to see if any of the other effects aren't working?

http://brianchirls.github.io/Seriously.js/examples/

@GanWeaving
Copy link

@brianchirls
animationwith seriously.go has the white dot problem (Chrome, Win7)

the others all seem to work fine (couldn't test those who required a camera)

@brianchirls
Copy link
Owner Author

Thanks for testing. That's good news.

The examples are based on the master branch, so the new version of the split effect is not in there yet. There are several other effects that have branching in the shaders (e.g. blend, accumulator, channels, displacement, etc.) but for some reason, they don't seem to be a problem for you. So I'm gonna close this one for now, and we'll keep an eye on the issue.

@scottgarner has been notified of the updated Chroma effect, and he can add it to the p5js site when he's ready. Both Chroma and Split will be updated with the next release of Seriously.js.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants