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

MultiplayerSynchronizer - "Watch" properties - on_delta_receive: Condition "props.size() == 0" is true. Returning: ERR_INVALID_DATA #78669

Closed
Federick732000 opened this issue Jun 25, 2023 · 3 comments · Fixed by #78709

Comments

@Federick732000
Copy link

Federick732000 commented Jun 25, 2023

Godot version

v4.1.beta2.mono.official [a2575cb]

System information

Godot v4.1.beta2.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated GeForce RTX 2060 () - AMD Ryzen 5 2600X Six-Core Processor (12 Threads)

Issue description

I'm using a Multiplayer Synchronizer to sync some variables of two instantiated players (each one has local multiplayer authority). Setting the "watch" property on the multiplayer synchronizer properties causes the player who receive the delta update to receive this error and to not store the updated value:

E 0:00:01:0604   on_delta_receive: Condition "props.size() == 0" is true. Returning: ERR_INVALID_DATA
  <C++ Source>   modules/multiplayer/scene_replication_interface.cpp:778 @ on_delta_receive()

I don't know if the problem is Mono-specific, but the error comes up even on Node3D default properties.

Steps to reproduce

  • Create a simple scene with a platform
  • Create a "Player" scene, that contains a MultiplayerSynchronizer, and put some properties to sync with the "watch" property ON and the "sync" property to OFF
  • Create a script to make one player create the server and the other to connect as a client to the server
  • Make the server instantiate the player on connect
  • Make a script to change the synced variables, to trigger the watched properties change

Minimal reproduction project

Test_delta.zip
Test_delta-gdscript.zip

@AThousandShips

This comment was marked as outdated.

@Federick732000
Copy link
Author

Understood. I converted the project to gdscript and the problem is still there. I added the project to the issue

@AThousandShips
Copy link
Member

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants