tr7zw / FirstPersonModel

Enables the third person Model in firstperson
MIT License
132 stars 25 forks source link

Cosmetica Issue while rendering head cosmetic (Along with TooManyPlayers incompat) #301

Open craftish37 opened 1 year ago

craftish37 commented 1 year ago

For every player skin, the head part isn't rendering AT ALL, causing this funny effect : image France in 1798...

Here's my logs, every mod should be up to date (maybe not the ones that are from CF) latest.log

tr7zw commented 1 year ago

Hm second report so far. Haven't been able to determine what exactly is causing that(which other mod).

craftish37 commented 1 year ago

Binary searching time so 👀 I'll let you know if I find anything 👍 (Also I didn't have this issue before)

tr7zw commented 1 year ago

The odd thing is, the update didn't change anything about the head logic. So no clue why it suddenly is an issue.

craftish37 commented 1 year ago

Ok it's not a mod thing, even tho I saw a little problem with cosmetica hats, nothing too complicated

Basically, on servers with a RP, the head disapears after loading the RP, which wasn't the case before

More related to the recent update, with a cosmetica hat on, we can see the hat on our head, but moving wrongly in 1st person : https://youtu.be/qZop_9ipVf0

tr7zw commented 1 year ago

Hm, that hat/helmet should not be visible at all. This is most likely a bug from cosmeticas end(ignoring the fabric api/player feature renderers/head position).

The other person also had the issue with some server RP, but I'm not sure how that could be connected(maybe some custom shader logic inside the RP?).

craftish37 commented 1 year ago

The server's RP is hosted on https\://download.mc-packs.net/pack/1dbd40d0a3306ad8f1aa9acf640595bb649f7fbe.zip Also I'll try to figure out if Wynncraft have any shader on too

tr7zw commented 1 year ago

With that resource pack loaded, still not having this issue(also putting stuff into the headslot).

tr7zw commented 1 year ago

Update: Figured it out. The latest version of Essential is incompatible.

craftish37 commented 1 year ago

The lack of head was just an incompat with TooManyPlayers, now gotta check for th cosmetica issue 👌