-
-
Notifications
You must be signed in to change notification settings - Fork 120
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
[ME] Projector support #244
[ME] Projector support #244
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall it seems fine, just a few concerning parts.
src/MaterialEditor.Core.Studio/Core.MaterialEditor.SceneController.cs
Outdated
Show resolved
Hide resolved
Here are the projectors for KK/KKS And those for HS/HS2/AI are included with RE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good, will merge after testing is done.
Projector support for ME. Formely you had to use Render Editor to edit them in games that had projectors. Koikatsu doesn't have them yet, but I'm porting those, but then RE doesn't exist on KKS.
All projector properties are treated as material properties in the UI, and thus grouped together with them. I did it like this to make editing them a little easier (grouped together + copy/pasting edits), and since projectors can only ever have one material anyway. Since projectors can only have one material, I also hide the "Copy Material" button now whenever the provided function for that button is
null
.Fow now I've made all character versions of the projector methods throw a
NotImplementedException
, since projectors don't seem to be used there at all. If the rest of the code is fine I will add those functions for completeness sake though (and in case they do exist there, though I doubt that)There is also the problem of potential conflicts with RE, This is because projectors share their material instance by default, while renderer materials do not. To circumvent this you need to assign a unique copy yourself and keep track if you've already assigned a unique copy or not. RE seems to do this, and might cause conflicts.
In my testing they don't conflict for some reason though, except for the
orthographic
setting, which can't be edited in ME if it's enabled in RE