Sodium Reloaded (Unofficial)

Sodium Reloaded (Unofficial)

0 Downloads

Show warning when NPOT textures are preventing the use of mip-mapping

TheNukeLore opened this issue ยท 5 comments

commented

Request Description

When resourcepacks or mods use non-power of two textures, it will drop down the mipmapping level, respectively. However, the only warning is a small warning in the log files; however, this greatly affects the visuals of the game.
I request that sodium could do a few potential things, or potentially all:

  • Gray out the mipmapping setting in the video settings alongside a description of why this has happened via a tooltip
  • Give a warning toast similar to the existing warning toasts sodium has regarding core-shaders

Should probably list out the packs and textures causing said issues as well.

commented

Note: I've been told that NeoForge (at least on 1.21?) actually allows NPOT textures to be used, even though it cannot correctly generate mipmaps for them. So we may want to do this only on Fabric for the time being.

commented

I'd support replacing any npot textures with the missing texture to force ppl to make their damn texture packs correctly

commented

Users don't have any choice in what nonsense their resource packs (or mods) are doing. The most that I'm comfortable with doing is showing a warning similar to the other ones that exist for incompatible resource packs, so that users know why there are problems.

commented

The warning is a good idea. Imo it should also appear after loading in because it's likely people already had the texture packs selected since long ago. The on-load warning should also identify any mods with bad textures and nicely list them for the player (but clearly list out texturepacks and mods separately for user convinience). A small note saying "Please report these issues to the owners of affected mods/texturepacks" so that'll hopefully bring about more pressure to fix those things.

The on-load warning should have a "don't show this again" checkbox, so that it doesn't get annoying.

Also with the warning there should be a small image showing the difference between disabled/enabled mipmapping so users know what they're missing out on (pick out the worst case scenario for mipmaps off like leaves). Majority of players do not know what a "mipmapping" is
Something like this:
image

commented

npot assets are already listed in the log (they also ''work as intended'' on fabric, but mip-mapping is broken globally). However the assets are not linked to the resource pack of origin (would be great to know, not always obvious otherwise)