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

[Bug - models] ifb() animation function doesn't exsist #274

Open
magmamir opened this issue Sep 26, 2024 · 3 comments
Open

[Bug - models] ifb() animation function doesn't exsist #274

magmamir opened this issue Sep 26, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@magmamir
Copy link

Describe the bug
It seems that the ifb() animation function doesn't exist in EMF, and using it in a .jem file will cause an error, disabling all resource packs. In EMF, its functionality appears to be handled by the usual if(). This is pretty convenient, but it breaks compatibility since Optifine unfortunately only allows the ifb() function to return boolean values.

Maybe it's possible to add an ifb() function with the functionality required by Optifine, while still allowing the regular if() to return boolean values? To avoid breaking existing resource packs designed specifically for EMF.

Versions:

  • Minecraft Version: 1.21
  • EMF version: 2.2.6
  • ETF version: 6.2.5
@magmamir magmamir added the bug Something isn't working label Sep 26, 2024
@magmamir magmamir changed the title [Bug - models] ifb() animation functions doesn't exsist [Bug - models] ifb() animation function doesn't exsist Sep 26, 2024
@Traben-0
Copy link
Owner

ifb does exist...

the reason if() might appear in the failure cause is it acts as a parent to ifb() to handle the initial boolean condition logic then ifb() handles the return as a boolean

ill need your log with this resourcepack failure in it and the model thats failing

@magmamir
Copy link
Author

magmamir commented Oct 3, 2024

Sorry for the delay
latest.log
resources.zip
iron_golem2 is the failing model

@BlackTalons
Copy link

BlackTalons commented Oct 7, 2024

Got the same issue in the forge version.

A workaround to convert the common if() to boolean:

"equals( if( condition, 0, 1), 0, 0)"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants