OnDeserialize failed Exception=System.IO.EndOfStreamException

[Stolichnayer]

Bug: OnDeserialize failed Exception=System.IO.EndOfStreamException (see below) object=PlayerPrefab(Clone) component=Mirror.NetworkAnimator sceneId=0 length=20

Reproduce: I'm building a FPS game where there is the main player's PlayerPrefab with two children like this: where:

• FirstPersonCharacter: FPS camera + Firearms that only local player can see
• Avatar : Full body avatar that all other players can see, except local player

NetworkTransform works great, all movement is being synchronized perfectly.

The exception is thrown when I add a NetworkAnimator component on PlayerPrefab and attach its child's Avatar Animator on the animator field.

The exception is not thrown when the Animator is on the same GameObject with the NetworkAnimator, for example both in the parent PlayerPrefab (as I can see it was fixed in #570 issue).

Screenshots

I also recorded a video of the bug to make clearer what I'm trying to do.

https://www.youtube.com/watch?v=Y7p6N_PyiKM

Desktop:

• OS: [Windows]
• Build target: [Desktop]
• Unity version: [2020.3.13f1]
• Mirror Version: [42.2.12 - July 12, 2021 from AssetStore]

[robtherobot12]

Yup, same problem here.

[miwarnec]

sorry, late reply. sounds like NetworkAnimator isn't always serializing the same amount of data.

just to be clear @robtherobot12 @Stolichnayer are you changing NetworkAnimator's .animator field at runtime, or no?

[robtherobot12]

This was a while ago, but no, I wasn’t touching that property.

On Sun, 29 Jan 2023 at 04:17, mischa @.***> wrote:

sorry, late reply. sounds like NetworkAnimator isn't always serializing the same amount of data.

just to be clear @robtherobot12 https://github.com/robtherobot12 @Stolichnayer https://github.com/Stolichnayer are you changing NetworkAnimator's .animator field at runtime, or no?

— Reply to this email directly, view it on GitHub https://github.com/MirrorNetworking/Mirror/issues/2852#issuecomment-1407557439, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHY4D6DKROCP5NVBR2EDZLDWUXVNJANCNFSM5BCGTP4A . You are receiving this because you were mentioned.Message ID: @.***>

[miwarnec]

@robtherobot12 thanks for the reply :) is it still happening with latest mirror, or can we close this?

[robtherobot12]

@vis2k I'll check today, and write if so.

[egorf710]

@Stolichnayer after experimenting, I realized that the error magically disappears if I move the network animator component to the very top to the network identity component in the hierarchy of components. the screenshots are more clear

[miwarnec]

looking into this again. it can only happen if OnSerialize and OnDeserialize read/write different amounts of data.

[miwarnec]

hey @Stolichnayer @robtherobot12 @egorf710 sorry for late reply. there's a pull request with a fix, which we will merge soon: https://github.com/MirrorNetworking/Mirror/pull/3711

this will 100% always serialize and deserialize the exact amount of data. and if parameters.count or layers.count is mismatching, it will show an exact error and let you know what went wrong.