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

chore: Increment package versions for NGO 1.0.0-pre.4 and update changelogs #1563

Merged
merged 3 commits into from
Jan 6, 2022

Conversation

ashwinimurt
Copy link
Contributor

@ashwinimurt ashwinimurt commented Jan 4, 2022

This is a backport of #1562

Changelog

com.unity.netcode.gameobjects

  • Changed: Updated Netcode for GameObjects dependency to 1.0.0-pre.4

@ashwinimurt ashwinimurt force-pushed the backport/update-package-versions-pre.4 branch from 00ceafc to 25d775e Compare January 4, 2022 22:57
@ashwinimurt ashwinimurt changed the title Increment package versions for NGO 1.0.0-pre.4 and update changelogs chore: Increment package versions for NGO 1.0.0-pre.4 and update changelogs Jan 4, 2022
@ashwinimurt ashwinimurt requested a review from 0xFA11 January 4, 2022 23:11
@ashwinimurt ashwinimurt force-pushed the backport/update-package-versions-pre.4 branch from 5a535c2 to c959aab Compare January 5, 2022 01:31
- Updated Netcode for GameObjects dependency to 1.0.0-pre.4 [#1562](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1562)
- Removed 'Maximum Packet Size' configuration field in the inspector. This would cause confusion since the maximum packet size is in effect always the MTU (1400 bytes on most platforms). [#1403](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1403)
- Updated com.unity.transport to 1.0.0-pre.10 [#1501](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1501)
- The 'Send Queue Batch Size' configuration field now controls the size of the send queue, rather than the size of a single batch of messages. Consequently, it should be set much higher than it was previously. [#1491](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1491)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should remove this line. It was part of an earlier iteration of that PR and is not correct anymore. I guess I forgot to remove it from the changelog when I backported the changes to release/1.0.0.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

- Updated Netcode for GameObjects dependency to 1.0.0-pre.4 [#1562](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1562)
- Removed 'Maximum Packet Size' configuration field in the inspector. This would cause confusion since the maximum packet size is in effect always the MTU (1400 bytes on most platforms). [#1403](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1403)
- Updated com.unity.transport to 1.0.0-pre.10 [#1501](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1501)
- The 'Send Queue Batch Size' configuration field now controls the size of the send queue, rather than the size of a single batch of messages. Consequently, it should be set much higher than it was previously. [#1491](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1491)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- The 'Send Queue Batch Size' configuration field now controls the size of the send queue, rather than the size of a single batch of messages. Consequently, it should be set much higher than it was previously. [#1491](https://github.com/Unity-Technologies/com.unity.netcode.gameobjects/pull/1491)

@ashwinimurt ashwinimurt force-pushed the backport/update-package-versions-pre.4 branch from c959aab to 61956de Compare January 6, 2022 00:21
@ashwinimurt ashwinimurt force-pushed the backport/update-package-versions-pre.4 branch from 61956de to 0ca9728 Compare January 6, 2022 00:25

### Added

- Added new 'Max Send Queue Size' configuration field in the inspector. This controls the size of the send queue that is used to accumulate small sends together and also acts as an overflow queue when there are too many in-flight packets or when other internal queues are full.
- Added new 'Max Send Queue Size' configuration field in the inspector. This controls the size of the send queue that is used to accumulate small sends together and also acts as an overflow queue when there are too many in-flight packets or when other internal queues are full. (#1491)
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm sorry to bother you again @simon-lemay-unity :)
This line should stay, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, this one is correct.

@0xFA11 0xFA11 enabled auto-merge (squash) January 6, 2022 14:34
@0xFA11 0xFA11 merged commit b14204c into release/1.0.0 Jan 6, 2022
@0xFA11 0xFA11 deleted the backport/update-package-versions-pre.4 branch January 6, 2022 15:30
mollstam pushed a commit to Keepsake-Games/com.unity.netcode.gameobjects that referenced this pull request Feb 13, 2023
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.

3 participants