Skip to content

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

Lilaa3 added 2 commits July 21, 2024 09:18
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
@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
@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