chore: merge release into master#649
Closed
mattwalsh-unity wants to merge 145 commits intomasterfrom
Closed
Conversation
- Moved the core MLAPI to the proper package structure (it has to be in this structor - Added examples of transport packages living in the same repo :D (ENET/Initial UTP) - Added basic Yamato package validation scripts and publish/promote
These where internal/exeperimental changes to the Transport API.
BREAKING CHANGE: clientId is required when defining a MoveValidationDelegate
…tiplayer.mlapi into develop
Updating the utp package readme
…ompletely. This caused an exception in the editor when trying to access the inspector because the SerializedProperty doesn't get initialized anymore.
…tUpdates Feature/fix remove max object updates
… some modifications Bringing over the DataStream class from UTP into the core repo, we remove some of the unused features as well as make it work with some already defined MLAPI types. This class is also optimized to be used in a job context/burst context and has no managed allocations, as well thread safety. (Original) https://github.com/Unity-Technologies/netcode/blob/master/com.unity.transport/Runtime/DataStream.cs
…rialization feat(serialization): Bringing over the DataStream class from UTP
* refactor: Fixed indents * fix: Fixed range interpolation
Our custom CI configuration started failing after a change in the upm-ci tool. This change updates the CI configuration to use the standard templates for multi-package repositories. As a side effect, it also introduces a test/sample/gym project where we can exercise the packages and put tests we don't want to ship with the packages themselves. Tests run against 2020.1 and trunk, on windows, mac, and ubuntu Pull Requests to "develop" automatically trigger pack, test, and validation of the mlapi package
…t_obj refactor: internally breaking, send no longer has client obj parameter
…ires ownership (#627) * add todo comments * fix: implement warning logs on non-owner invoking ServerRpc that requires ownership
* test: NetworkTickSystem test * test: NetworkTickSystem test, removed unneeded code
…#629) * fix: Fix object traversal assigning instanceIds to none scene objects * comment added Co-authored-by: Matt Walsh <matt.walsh@unity3d.com>
If something causes an exception it's possible that the profiler never allocates data. in these cases, there is no point in sending data outside of MLAPI
* Add release notes to changelog * docs: added release notes to changelog * Update com.unity.multiplayer.mlapi/CHANGELOG.md Co-authored-by: Luke Stampfli <43687322+LukeStampfli@users.noreply.github.com> * update per review for bitserializer and networkserializer * revised per feedback - API refactor * update remove bitserializer * renaming * Update com.unity.multiplayer.mlapi/CHANGELOG.md Co-authored-by: M. Fatih MAR <mfatihmar@gmail.com> * Update com.unity.multiplayer.mlapi/CHANGELOG.md * Update com.unity.multiplayer.mlapi/CHANGELOG.md * added known issues and scene mgmt fix * revisions per review * profiler final - no more updates * format Co-authored-by: Luke Stampfli <43687322+LukeStampfli@users.noreply.github.com> Co-authored-by: Matt Walsh <69258106+mattwalsh-unity@users.noreply.github.com> Co-authored-by: M. Fatih MAR <mfatihmar@gmail.com>
- Mirror package readme.md in root readme.md - Add repo directory structure description - Remove contributing and code of conduct from package directory
…icks (#640) PerformanceDataManager reuses PerformanceTickData by resetting per-frame Replaces deferred linq query in PerformanceTickData loop.
… missed in the profiler (#641) Co-authored-by: Matt Walsh <69258106+mattwalsh-unity@users.noreply.github.com>
… clean import (#645) * Add an dummy ILPP to force 2019.4 to proper compilation order * Turning on this test for 2019.4 * Hack -> Workaround rename Co-authored-by: Matt Walsh <matt.walsh@unity3d.com> Co-authored-by: Matt Walsh <69258106+mattwalsh-unity@users.noreply.github.com>
LukeStampfli
suggested changes
Mar 22, 2021
Contributor
LukeStampfli
left a comment
There was a problem hiding this comment.
Just looked at repo documentation files and structure.
- There is both a
LICENSEandLICENSE.mdin the root and aLICENSE.mdfile in the package folder. Should remove the duplicate. I think for both the root and the package LICENSE file they shouldn't be .md files. LICENSE should be a raw text file (no file ending) to guarantee that it is displayed the same for everyone. - Readme, contributing, CoC, Changelog, third party notices look good.
com.unity.multiplayer.transport.utpis not finished or usable right? Why is it in release in the first place?
Contributor
|
superseded by #652 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.