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

WebGL Multiple Render Targets (DrawBuffers) #9358

Closed
wants to merge 16 commits into from
Closed

Conversation

edankwan
Copy link

Resolved the conflict of @mattdesl 's pull request: #8439

The only change I made in the example is that I use Float type on both diffuse and normal attachments because using different types doesn't seem to be working due to the issue 5 in https://www.khronos.org/registry/gles/extensions/EXT/EXT_draw_buffers.txt

@edankwan
Copy link
Author

edankwan commented Jul 19, 2016

@edankwan
Copy link
Author

edankwan commented Aug 1, 2016

I just made an update for the MultiRenderTarget with the new es6 rollup styles.

@WestLangley
Copy link
Collaborator

@edankwan Can you please remove the builds from the PR?

@edankwan
Copy link
Author

edankwan commented Aug 2, 2016

Removed

@edankwan
Copy link
Author

Any news on this merge?

@edankwan
Copy link
Author

.

@edankwan
Copy link
Author

edankwan commented Sep 4, 2016

Resolve new conflicts again... and updated the preview demo link above.

@mrdoob
Copy link
Owner

mrdoob commented Sep 8, 2016

Could you remove the builds from this PR? I think that's why the conflicts keep coming back.

@edankwan
Copy link
Author

edankwan commented Sep 8, 2016

Done. Sorry about it.

@mrdoob
Copy link
Owner

mrdoob commented Sep 11, 2016

Hmm, can you try pulling/merging from dev?

@edankwan
Copy link
Author

Merged and pushed. It seems working just fine: http://edankwan.com/tests/threejs-multi-render-target/webgl_mrt.html

@edankwan
Copy link
Author

Any news?

@edankwan
Copy link
Author

Merged and removed the built files again.

@mrdoob
Copy link
Owner

mrdoob commented Oct 29, 2016

#9965

@Mugen87 Mugen87 closed this Apr 8, 2017
@mattdesl mattdesl mentioned this pull request Dec 14, 2017
@takahirox
Copy link
Collaborator

@mrdoob Would you merge MRT feature if I (or someone) clean up this PR? Or should we wait til WebGL2Renderer is available?

@WestLangley
Copy link
Collaborator

Would you merge MRT feature if I (or someone) clean up this PR?

I would like to see this implemented, too.

@gkjohnson
Copy link
Collaborator

Is it worth resurrecting this PR now that WebGL2 is available and supports MRT? I'd really like to see this for use with deferred rendering.

@takahirox
Copy link
Collaborator

takahirox commented May 4, 2019

If you folks don't mind I'll try to clean up this PR because this PR isn't updated for years and I'd like to see MRT in Three.js. Let me know if anyone already started or wants to try, I don't wanna be conflicted with.

@takahirox takahirox mentioned this pull request May 5, 2019
@takahirox
Copy link
Collaborator

Created PR #16390

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.

6 participants