Skip to content

fix: Realtime Network Stats Monitor Not Displaying RPC Metrics in Release Builds #2980

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

Merged

Conversation

NoelStephensUnity
Copy link
Collaborator

Adding UNITY_MP_TOOLS_NET_STATS_MONITOR_ENABLED_IN_RELEASE to all of the various dependencies required to enable RNSM tracking of RPC messages.

MTTB-30

Changelog

  • Fixed: Issue where the real-time network stats monitor was not able to display RPC traffic in release builds due to those stats being only available in development builds or the editor.

Testing and Documentation

  • No tests have been added (requires manual testing) but verified in Boss Room release build.
  • No documentation changes or additions were necessary.

Adding UNITY_MP_TOOLS_NET_STATS_MONITOR_ENABLED_IN_RELEASE to all of the various dependencies required to enable RNSM tracking of RPC messages.
adding change log entry
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review July 19, 2024 18:31
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner July 19, 2024 18:31
Copy link
Collaborator

@fluong6 fluong6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issue is fixed with this PR.

@NoelStephensUnity NoelStephensUnity enabled auto-merge (squash) July 19, 2024 18:55
@NoelStephensUnity NoelStephensUnity merged commit 4400390 into develop Jul 19, 2024
23 of 24 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/rnsm-rpc-stats-not-present-in-release-builds branch July 19, 2024 19:13
NoelStephensUnity added a commit that referenced this pull request Jul 19, 2024
This fix includes the up-port of #2980 (RNSM not tracking RPCs in release builds).
Fixing the issue where I forgot to add the serialized type for the not optimized warning message update.
NoelStephensUnity added a commit that referenced this pull request Jul 22, 2024
…ge handling (#2979)

* update

Adding xml API documentation and tooltips for the `NetworkConfig.NetworkMessageMetrics` and `NetworkConfig.NetworkProfilingMetrics` properties.

* update

Added a static helper method in `NetworkManager` to handle logging when a serialized type is not yet optimized for distributed authority.

* fix

This fix includes the up-port of #2980 (RNSM not tracking RPCs in release builds).
Fixing the issue where I forgot to add the serialized type for the not optimized warning message update.
NoelStephensUnity added a commit that referenced this pull request Jul 29, 2024
* update

Updating NGO to v2.0.0-pre.3
Updating UTP dependency to v2.3.0
Updating manifest to current dependencies
Updating project version to 6000.0.10f1

* chore: 2.0.0-pre.3 additional last minute updates (#2978)

* update

missed actually updating the version.

* update

Added `UnityTransport.GetNetworkDriver` and `UnityTransport.GetLocalEndpoint` methods.

* update

Updating changelog file.

* fix: metrics tooltips and serialized type non optimized warning message handling (#2979)

* update

Adding xml API documentation and tooltips for the `NetworkConfig.NetworkMessageMetrics` and `NetworkConfig.NetworkProfilingMetrics` properties.

* update

Added a static helper method in `NetworkManager` to handle logging when a serialized type is not yet optimized for distributed authority.

* fix

This fix includes the up-port of #2980 (RNSM not tracking RPCs in release builds).
Fixing the issue where I forgot to add the serialized type for the not optimized warning message update.

* fix: deferred despawn gc allocation (#2983)

* fix

Remove GC alloc when creating list for deferred despawn.

* update

adding log entry for fix.

* update

adding PR for the fix

* chore: Anticipated not supported warning (#2982)

chore

notify user that this component is not supported in distributed authority mode

* fix: allow clients to set client synchronization mode distributed authority (#2985)

* fix

Allow clients to set client synchronization mode when in distributed authority mode since clients can be promoted to session owner.

* test

apply same rules for the integration test version of the scene handler

* update

adding change log entry

* chore

making a new unreleased changelog entry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants