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

[F3D] Edge case for decals, use hashed for cutout materials instead of blend + more #401

Merged
merged 18 commits into from
Aug 20, 2024

Conversation

Lilaa3
Copy link
Collaborator

@Lilaa3 Lilaa3 commented Aug 5, 2024

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

@Lilaa3 Lilaa3 changed the title [F3D] Edge case for decals, use hashed for cutout and xlu materials instead of blend + more [F3D] Edge case for decals, use hashed for cutout materials instead of blend + more Aug 7, 2024
@Lilaa3
Copy link
Collaborator Author

Lilaa3 commented Aug 9, 2024

This should also simplify this function in f64render along with render mode flags parsing

Copy link
Contributor

@Yanis42 Yanis42 left a 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 Show resolved Hide resolved
fast64_internal/f3d/f3d_material.py Outdated Show resolved Hide resolved
Comment on lines 197 to 198
if settings.rendermode_advanced_enabled and settings.set_rendermode:
# Already in advanced mode, don't overwrite this with the preset
Copy link
Contributor

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?

Copy link
Collaborator Author

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?

fast64_internal/f3d/f3d_material.py Outdated Show resolved Hide resolved
fast64_internal/oot/f3d/properties.py Outdated Show resolved Hide resolved
@Dragorn421
Copy link
Contributor

btw why not alpha clip for cutout materials

@Lilaa3
Copy link
Collaborator Author

Lilaa3 commented Aug 11, 2024

btw why not alpha clip for cutout materials

sure i can do that for pre 4.2, wont make a visual difference tho

@Lilaa3
Copy link
Collaborator Author

Lilaa3 commented Aug 13, 2024

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

@Lilaa3 Lilaa3 requested a review from Dragorn421 August 15, 2024 23:40
@Dragorn421 Dragorn421 added the merge soon Will be merged in a few days at most if nothing else comes up label Aug 19, 2024
@Dragorn421 Dragorn421 merged commit 7bb8e71 into Fast-64:main Aug 20, 2024
1 check passed
@Lilaa3 Lilaa3 deleted the use-dithered-for-cutout branch August 24, 2024 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge soon Will be merged in a few days at most if nothing else comes up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants