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

Merged output from wig-based mods is has wrong head part data with dewiggifier off #164

Closed
focustense opened this issue Nov 5, 2022 · 0 comments
Labels
bug Something isn't working easynpc Issues/requests relating the EasyNPC app

Comments

@focustense
Copy link
Owner

Surfaced recently on Discord, when a user was trying to merge High Poly NPC Overhaul (2.0, where all the hairs are wigs). The wig conversion option was turned off, and the facegen files had no hair (as expected), but the merged plugin was missing the HighPoly_HairBald part and instead had hair.

Cause is that the DewiggifyFacegensTask honors the wig conversion setting, but the DewiggifyRecordsTask does not honor the setting and always runs regardless. Thus, counterintuitively, not enabling wig conversion is not only riskier, but guaranteed to cause issues with these mods.

Solution is simple, just fix the task to follow the setting. Also, this setting should really be disabled by default since it was always an experimental feature, unsure why it's on by default.

@focustense focustense added bug Something isn't working easynpc Issues/requests relating the EasyNPC app labels Nov 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easynpc Issues/requests relating the EasyNPC app
Projects
None yet
Development

No branches or pull requests

1 participant