-
Notifications
You must be signed in to change notification settings - Fork 73
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
[F3D] Edge case for decals, use hashed for cutout materials instead of blend + more #401
Conversation
This should also simplify this function in f64render along with render mode flags parsing |
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.
can't say I understand what this does but here's my tiny contribution to the review
fast64_internal/f3d/f3d_material.py
Outdated
if settings.rendermode_advanced_enabled and settings.set_rendermode: | ||
# Already in advanced mode, don't overwrite this with the preset |
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.
can you update the comment to explain what happens wrt set_rendermode
afaict you're basically implementing material bleed from world defaults in the props. Wouldn't this be simpler by just setting the preset to the world defaults preset if set_rendermode is unchecked?
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.
Could you give me feedback on if the comments are good now?
btw why not alpha clip for cutout materials |
sure i can do that for pre 4.2, wont make a visual difference tho |
Currently fast64 main is also assuming fog when no rendermode is set btw, this was mostly hidden before i removed the requirement for the geo mode to activate the fog blender (i say mostly because it caused unset rendermode sm64 materials to assume the fog blender was on even without setting the rendermode lol), it only showed now but this pr accidentally fixes it |
should´t really help anything but its one less node for pre 4.2 users also change hashed to dithered
Main change is the title, but I also updated rendermode_preset_to_advanced to actually update the settings based on layer, not only that but the world defaults are used for layers now