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

WebGPURenderer: Support to .backgroundNode #24213

Merged
merged 8 commits into from
Jun 9, 2022
Merged

Conversation

sunag
Copy link
Collaborator

@sunag sunag commented Jun 9, 2022

@sunag
Copy link
Collaborator Author

sunag commented Jun 9, 2022

/ping @mrdoob

@sunag sunag marked this pull request as ready for review June 9, 2022 04:39
@sunag sunag added this to the r142 milestone Jun 9, 2022
@mrdoob
Copy link
Owner

mrdoob commented Jun 9, 2022

Looking good!

@mrdoob
Copy link
Owner

mrdoob commented Jun 9, 2022

Hmm... Shouldn't the helmet go white when the brightness is set to 1 too?

Screen Shot 2022-06-09 at 3 01 25 PM

@sunag
Copy link
Collaborator Author

sunag commented Jun 9, 2022

Hmm... I will remove the brightness because it seems to have a conflict between physical and non-physical properties here.

@@ -41,6 +46,38 @@ class WebGPUBackground {
_clearAlpha = 1;
forceClear = true;

} else if ( background.isNode === true ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we actually create BackgroundNode, which will do this (at least creating a mesh and aligning it to the camera) by itself? I think it will allow to more easily implement in the future the Background suggestion.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That sounds interesting...

@sunag sunag merged commit f30599e into mrdoob:dev Jun 9, 2022
abernier pushed a commit to abernier/three.js that referenced this pull request Sep 16, 2022
* beautify matrix columns

* add support to .onBeforeRender()

* add support to .backgroundNode

* add .backgroundNode to examples

* cleanup

* cleanup

* remove brightness, rename contrast to intensity

* fix get empty image[0]
snagy pushed a commit to snagy/three.js-1 that referenced this pull request Sep 21, 2022
* beautify matrix columns

* add support to .onBeforeRender()

* add support to .backgroundNode

* add .backgroundNode to examples

* cleanup

* cleanup

* remove brightness, rename contrast to intensity

* fix get empty image[0]
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