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] Show tlut othermode in materials and world defaults #385

Merged
merged 6 commits into from
Aug 6, 2024

Conversation

Lilaa3
Copy link
Collaborator

@Lilaa3 Lilaa3 commented Jul 21, 2024

The mode is still automatically set when using CI textures, however the default can now be changed and it hides what's happening under the hood just a bit less, I also removed some code from f3d_bleed, specifically I removed reset_command_dict (unused) and DPSetTextureLUT from the cmd list in add_reset_cmd because it is always set by SPSetOtherMode like other othermodes.
image
image

The mode is still automatically set when using CI textures, however the default can now be changed and it hides what's happening under the hood just a bit less,  I also removed some code from f3d_bleed, specifically I removed reset_command_dict (unused) and DPSetTextureLUT from the cmd list in add_reset_cmd because it is always set by SPSetOtherMode like other othermodes.
@jesusyoshi54
Copy link
Collaborator

If your texture is not set, then you should be able to set the value of the TextureLUT property. This would make it so that the TextureLUT property is properly reverted to normal in the case where a texture is loaded via some function node. This should also fix the bug where unset textures would get defaulted to G_TT_NONE while using the mat write method write_all (bleed and DL exporter option) or if you set the default to use LUT and had the unset texture apply G_TT_NONE

And update other_mode_h_to_dict for the future
@Lilaa3 Lilaa3 requested a review from jesusyoshi54 July 21, 2024 18:27
@Lilaa3 Lilaa3 mentioned this pull request Jul 25, 2024
7 tasks
fast64_internal/f3d/f3d_material.py Show resolved Hide resolved
fast64_internal/f3d/f3d_material.py Outdated Show resolved Hide resolved
fast64_internal/f3d/f3d_material.py Outdated Show resolved Hide resolved
fast64_internal/f3d/f3d_material.py Show resolved Hide resolved
@Lilaa3 Lilaa3 added the f3d Has to do with the "f3d" code common to all games label Aug 2, 2024
@jesusyoshi54 jesusyoshi54 added the merge soon Will be merged in a few days at most if nothing else comes up label Aug 4, 2024
@Dragorn421 Dragorn421 merged commit 886fbf9 into Fast-64:main Aug 6, 2024
1 check passed
@Lilaa3 Lilaa3 deleted the show-tlut-mode branch August 24, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f3d Has to do with the "f3d" code common to all games 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