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

Regression in r127dev MultiRenderTarget GL_INVALID_OPERATION: Blit feedback loop: the read and draw framebuffers are the same. #21517

Closed
arpu opened this issue Mar 25, 2021 · 7 comments · Fixed by #21518

Comments

@arpu
Copy link

arpu commented Mar 25, 2021

r127dev
https://raw.githack.com/mrdoob/three.js/dev/examples/webgl2_multisampled_renderbuffers.html

works on stable r126
https://threejs.org/examples/?q=webgl2_multisampled_renderbuffers.#webgl2_multisampled_renderbuffers
To Reproduce

Live example

Expected behavior

should work on r127dev

Screenshots
Bildschirmfoto von 2021-03-25 15-06-37

Platform:

  • Device: [Desktop]
  • OS: [ Linux]
  • Browser: [Chrome 89]
  • Three.js version: [dev]
@arpu arpu changed the title Regrission in r127dev MultiRenderTarget GL_INVALID_OPERATION: Blit feedback loop: the read and draw framebuffers are the same. Regression in r127dev MultiRenderTarget GL_INVALID_OPERATION: Blit feedback loop: the read and draw framebuffers are the same. Mar 25, 2021
@Mugen87
Copy link
Collaborator

Mugen87 commented Mar 25, 2021

Good catch! It seems the E2E tests are not yet able to detect this issue.

@arpu
Copy link
Author

arpu commented Mar 25, 2021

i see this problem on testing MultiRenderTarget on webgl2 for MSSA >4 would be nice to add this to the example? with gl.MAX_SAMPLES

@mrdoob
Copy link
Owner

mrdoob commented Mar 25, 2021

i see this problem on testing MultiRenderTarget on webgl2 for MSSA >4 would be nice to add this to the example? with gl.MAX_SAMPLES

Would you like to do a PR for it?

@arpu
Copy link
Author

arpu commented Mar 25, 2021

i have some problems on this, i always get 0 Samples used in Spector.js

@arpu
Copy link
Author

arpu commented Mar 25, 2021

would be nice to get some hints how i can use this
testing on https://jsfiddle.net/xgw4705v/13/

@arpu
Copy link
Author

arpu commented Mar 25, 2021

looks like i need this #16895 to get this done

@mrdoob
Copy link
Owner

mrdoob commented Mar 25, 2021

Feel free to do a new PR to replace #16895 👍

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

Successfully merging a pull request may close this issue.

3 participants