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

fix anisotropy rotation #27153

Merged
merged 2 commits into from
Nov 9, 2023
Merged

fix anisotropy rotation #27153

merged 2 commits into from
Nov 9, 2023

Conversation

elalish
Copy link
Contributor

@elalish elalish commented Nov 9, 2023

Fixes #26967

Description

I had tested this so carefully, but it turns out I was backwards because I didn't realize the model I was using had UV coordinates that were inverted. Just a simple sign change fixed it.

Copy link

github-actions bot commented Nov 9, 2023

📦 Bundle size

Full ESM build, minified and gzipped.

Filesize dev Filesize PR Diff
656.6 kB (162.7 kB) 656.6 kB (162.7 kB) +0 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Filesize dev Filesize PR Diff
449.1 kB (108.6 kB) 449.1 kB (108.6 kB) +0 B

@elalish
Copy link
Contributor Author

elalish commented Nov 9, 2023

Hmm, it's down to 0.4% difference from 3.5% - I wonder if this one is just more sensitive to small GPU variations? What's the process here @mrdoob?

@mrdoob
Copy link
Owner

mrdoob commented Nov 9, 2023

I think that particular test is flakey.

@mrdoob mrdoob added this to the r159 milestone Nov 9, 2023
@mrdoob mrdoob merged commit 8f7ae2c into mrdoob:dev Nov 9, 2023
17 of 20 checks passed
@elalish elalish deleted the anisotropyRotation branch November 9, 2023 16:39
@LeviPesin
Copy link
Contributor

What's the process here @mrdoob?

The example should be added to the exceptions list then.

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.

Anisotropy rotation appears incorrect (clockwise)
3 participants