Armored Elytra

Armored Elytra

36k Downloads

Occasional incompatibility with quilt

SilverAndro opened this issue · 6 comments

commented

crash-2023-03-19_00.40.02-client.txt

Because armored elytra and QSL inject into the same place, theres a roughly 50/50 chance based on mod load order that, on quilt, will result in a crash. The issue is that your armor feature renderer may try to render elytra before quilt can save into a local variable what entity is being rendered (for its armor rendering API).

This can be remedied by either moving your rendering injection to be later in the method, or possibly by simply increasing the priority value, so that mixin sorts it to come after, although im less sure on that, you may have to lower the priority value instead.

commented

Awesome, thanks for the quick response!

commented

Unfortunately quilt is currently not supported, but I have plans to add support in the future and when I do so I will fix this.

commented

Any updates on this?

commented

Quilt support is still planned, I'll probably release that along with the 1.20 updates.
I can always backport Quilt support to other versions.

commented

Turns out the priority needs to be a smaller number for it to work

commented

image
1.20 coming eventually, fixing this exact issue now lol