Skip to content

Conversation

@Glorou
Copy link
Contributor

@Glorou Glorou commented Sep 22, 2025

There was an issue where non-vanilla material names would cause this function to a) fault because the material string was too short and would cause one of the functions to break, or b) return garbage since they dont conform to vanilla naming conventions (most of the time). This rewrite uses regex to check for the syntax safely in one regex match. I confirmed the function works using Penumbra and passing various material names and monitoring behavior.

@NotAdam NotAdam merged commit c0224d9 into NotAdam:master Sep 22, 2025
2 checks passed
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.

2 participants