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

Examples: Clean up #29209

Merged
merged 2 commits into from
Sep 17, 2024
Merged

Examples: Clean up #29209

merged 2 commits into from
Sep 17, 2024

Conversation

WestLangley
Copy link
Collaborator

The arguments were swapped.

darkness is in [0, 1].

@WestLangley WestLangley added this to the r168 milestone Aug 22, 2024
@WestLangley
Copy link
Collaborator Author

@Mugen87 I am unable to successfully create a screenshot for some reason. Can you please give it a try?

@Mugen87
Copy link
Collaborator

Mugen87 commented Aug 22, 2024

I wasn't able to push to your branch but here is the updated screenshot. I hope GitHub does not further compress the image.

webgl_postprocessing_advanced

@WestLangley
Copy link
Collaborator Author

Thanks, @Mugen87, but e2e is failing... Trying to figure out what is going on. :/

@mrdoob mrdoob modified the milestones: r168, r169 Aug 30, 2024
@WestLangley
Copy link
Collaborator Author

@Mugen87 You screenshot may be compressed. It is not passing e2e. And my screenshots are black.

I wasn't able to push to your branch

Not sure why that is, either. It says "Maintainers are allowed to edit this pull request."

@Mugen87
Copy link
Collaborator

Mugen87 commented Sep 17, 2024

Now it works! I guess I was on the wrong branch, not dev-clean_up.

@Mugen87 Mugen87 merged commit 1116706 into mrdoob:dev Sep 17, 2024
11 checks passed
@WestLangley WestLangley deleted the dev-clean_up branch September 17, 2024 16:32
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