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

perf: optimize InternalAutoFreezeMeaninglessBlendShapeProcessor #1325

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

bdunderscore
Copy link
Contributor

This saves over 250ms on my main avatar.

@bdunderscore bdunderscore marked this pull request as ready for review November 4, 2024 03:07
Copy link
Owner

@anatawa12 anatawa12 left a comment

Choose a reason for hiding this comment

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

I didn't know NativeDisableContainerSafetyRestriction.
I thought it's impossible to write same position from multiple jobs.

Thank you.

CHANGELOG.md Outdated
@@ -131,6 +131,7 @@ The format is based on [Keep a Changelog].
- Fix non-VRChat project support `#1310`
- 'shader' doesn't have a float or range property 'prop' error `#1312`
- Error if all components are on inactive GameObject `#1318`
- Improved performance in InternalAutoFreezeMeaninglessBlendShapeProcessor `#1325`
Copy link
Owner

Choose a reason for hiding this comment

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

Can you add PR number to Performance Improvements in Changed section instead of adding new entry?

@anatawa12
Copy link
Owner

I'll ignore the commitlint failure for this PR.
Could you check commit message before applying the suggestion in the future PRs, please?

Copy link
Owner

@anatawa12 anatawa12 left a comment

Choose a reason for hiding this comment

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

LGTM Thank you!

@anatawa12 anatawa12 merged commit 43ca9ef into anatawa12:master Nov 4, 2024
8 of 9 checks passed
@bdunderscore
Copy link
Contributor Author

I'll ignore the commitlint failure for this PR. Could you check commit message before applying the suggestion in the future PRs, please?

Ah, I thought you would squash it on merge. I’ll check that in the future.

@bdunderscore bdunderscore deleted the meaningless-perf branch November 4, 2024 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants