MarkusBordihn / BOs-Player-Companions

Player Companions adds additional NPC which will follow, protect or help you...
Other
2 stars 5 forks source link

Mobs without waiting sound causing crashes. #28

Closed M25R4PTOR closed 8 months ago

M25R4PTOR commented 8 months ago

Describe the bug Crash Game Like https://github.com/MarkusBordihn/BOs-Player-Companions/issues/27 but its rooster, samurai, fairy and raptor. I used version player_companions_1.20.1-6.6.2

To Reproduce Steps to reproduce the behavior:

  1. Open World LAN
  2. Mobs friends stay sit
  3. Wait and the game crash

Expected behavior No crash xD

Debugging Server(SinglePlayer Open World Lan) logs: crash-2023-10-12_03.41.21-server.txt crash-2023-10-12_04.23.46-server.txt crash-2023-10-12_05.05.54-server.txt crash-2023-10-12_05.53.24-server.txt

At the last minute the game crashed only in the client, I don't know if "Player Companions" has anything to do with it, this is my friend's log: Client(My friend on the same network) log: crash.log

Additional I just saw that you updated to version player_companions_1.20.1-6.7.0. This Friday/Saturday I will test the mod again with this version

Thanks for everything!

MarkusBordihn commented 8 months ago

Thanks a lot for the valuable feedback and the additional crash logs.

According the crash log the sound pitch is "null" which should be never the case, because it's a random generated number for all player companions. 🤔 I assume that some of your mods or resource packs change / adjust the sounds which leads to this crashes.

Anyways, I added some additional checks in the code to make sure that none of the sound related values will be used if they are not defined or overwritten by some other mods with an invalid value.

The updated version 6.7.1 should solve all related issues, but please let me know if you still experience any crashes with this version.

Thanks a lot.

M25R4PTOR commented 8 months ago

Is version 1.20 compatible with 1.20.1?

M25R4PTOR commented 8 months ago

I test player_companions_1.20-6.7.1 in minecraft 1.20.1 and crash with Dobutsu and Small-Slime

Debugging crash-2023-10-12_21.03.40-server.txt crash-2023-10-12_21.43.23-server.txt

Tomorrow and the next day i will be able to continue testing if you update the version

MarkusBordihn commented 8 months ago

Thanks for the feedback. Sorry, that's my fault, I forgot to add @override for the standard playSound, so this should be fixed with 6.7.2 and all sounds should now have the additional checks in place.

It could take some time until the new version is online, because of the CurseForge malware scanner which sometimes requires a manually human review which delays the release.

M25R4PTOR commented 8 months ago

Perfect, if version 6.7.2 is uploaded tonight in curseforge I will test it.

Is there a way to transfer companions from one world to another? I tried it with the structure block by putting the player companion item in a chest but in the new world the item shows an error when used.

M25R4PTOR commented 8 months ago

This is the first time it has taken so long for the version to appear in curseforge, is this normal? (Tonight I can test)

MarkusBordihn commented 8 months ago

Yes this is normal with their latest change. The have a malware scanner in place, when the malware scanner failed for some reason they need a manually human review instead which could take between 1-5 days.

See: image

Unfortunately there is nothing which I could do from my side to speedup this manually review.

In general it's good that they have something like this in place to make sure no malware is distributed over their network, but it's adds an additional delay for releases especially before/over the weekend.

M25R4PTOR commented 8 months ago

No problem, when it appears on the page I will test it.

Is there a way to transfer the companions obtained in one world to another by passing some type of file or with some mod?

Thanks for everything!

MarkusBordihn commented 8 months ago

Never tried this myself so far, but basically there are two possibile ways for this.

image

The backup files are stored in the world folder under player_companions.

M25R4PTOR commented 8 months ago

I have finally been able to test the version and it works perfectly.

If you create a new mod I would like to play it hehehe

Thank you for everything!