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

Shaders detecting Iris to disable features on Optifine #791

Open
3 tasks done
leumasme opened this issue Dec 14, 2024 · 3 comments
Open
3 tasks done

Shaders detecting Iris to disable features on Optifine #791

leumasme opened this issue Dec 14, 2024 · 3 comments
Labels
low prio | enhancement New feature or request that no dev has picked up yet

Comments

@leumasme
Copy link

Your GTNH Discord Username

leumasme

Mod Version

1.0.0-beta25

Bug Report

It seems like some shaders like Compementary Unbound check if they're running in Optifine or Iris via #if defined IS_IRIS ....
If it's not running in Iris and shader settings are set to enable features not supported by optifine like colored lighting, the shader refuses to work properly.
By manually patching the check out of a shader, I can see that Angelica seems to support colored lighting, and I'd imagine its nearly fully compatible since it's based on iris?
image
Would it be a good idea to define IS_IRIS for shader code to stop shaders from locking angelica users out of features?
In that case, probably also set another flag that mods can use to tell angelica apart from iris, and detect iris explicitly. Hmm i'm getting some user-agent header vibes... must've been the wind

Java Version

Java 21

Graphics Card Vendor

NVidia

Shaders

Yes: Please put relevant information about shaderpack in the report(what shaderpack, any special settings of note, etc)

MCPF or other features disabled by default

No: Go ahead and create an issue

Mod List or GTNH Pack Version

GTNH 2.7.0

Final Checklist

  • I have searched the issues and haven't found a similar issue.
  • I have read the known incompatibilities and this is not related to one of those.
  • I am running an officially released version. (Or, if I've compiled it myself I plan to fix the issue)
@ah-OOG-ah
Copy link
Member

We've talked to the Comp. devs, and defined an IS_ANGELICA tag for them to use. We don't currently define IS_IRIS because we don't fully support Iris's features at the time that flag was made, although we might be able to add it as an experimental setting.

@ah-OOG-ah ah-OOG-ah added the low prio | enhancement New feature or request that no dev has picked up yet label Dec 14, 2024
@mitchej123
Copy link
Collaborator

One thing one of the Euphoria patch devs suggested was looking at various feature flags that Iris introduced and reporting those as they are supported. IS_IRIS might be worth a try as an optional setting defaulting to Off, I'd say.

@mitchej123
Copy link
Collaborator

Also we're forked off of 1.16 iris, and I believe 1.18 iris is when they introduced the IS_IRIS flag

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low prio | enhancement New feature or request that no dev has picked up yet
Projects
None yet
Development

No branches or pull requests

3 participants