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

Protection level for Texture2D variable defined in TransferFunction.cs #267

Merged

Conversation

kvijaykiran
Copy link
Contributor

@kvijaykiran kvijaykiran commented Oct 8, 2024

Changed the protection level for Texture2D variable defined in TransferFunction.cs from private to protected. This change will allow for making 3rd party changes or customizations to transfer function creation as well as texture processing.

Without this change, I can only create a scriptable instance of TransferFunction in my own implementation, but Texture2Ds that I define and make changes to, do not get propagated to UnityVolumeRendering's workflow. I see a GetTexture() but did not find any SetTexture() that allows injecting my own texture.

Using a protected Texture2D variable will allow for inheriting the variable from TransferFunction class.

#266

…erFunction.cs from private to protected. This change will allow for making 3rd party changes or customizations to transfer function creation as well as texture processing.
@mlavik1 mlavik1 merged commit 0d45b4f into mlavik1:master Oct 17, 2024
@mlavik1
Copy link
Owner

mlavik1 commented Oct 17, 2024

Thank you for the PR! :)

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