Skip to content

Conversation

surfnerd
Copy link
Contributor

Merge changes from release 13 to master

Chris Elion and others added 20 commits February 16, 2021 22:56
* improved settings and move to default_settings

* update models
- updated release tag validation script to automate the updating of files with release tags that need to be changed as part of the pre-commit operation.
Co-authored-by: Philipp Siedler <p.d.siedler@gmail.com>
… (#4968)

Co-authored-by: Marwan Mattar <marwan@unity3d.com>
Co-authored-by: Chris Elion <chris.elion@unity3d.com>
@surfnerd surfnerd self-assigned this Feb 24, 2021
@@ -43,6 +47,7 @@ internal class RpcCommunicator : ICommunicator
Dictionary<string, ActionSpec> m_UnsentBrainKeys = new Dictionary<string, ActionSpec>();


#if MLA_SUPPORTED_TRAINING_PLATFORM
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this was the merge problem I was worried about. I think you can either remove all the #if MLA_SUPPORTED_TRAINING_PLATFORM except the top-level one, or just keep master the master version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

should be fixed now

Copy link
Contributor

@chriselion chriselion left a comment

Choose a reason for hiding this comment

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

Looks good (modulo the RpCommunicator merge)

@surfnerd surfnerd merged commit 2e50682 into master Feb 24, 2021
@surfnerd surfnerd deleted the release_13_branch-to-master branch February 25, 2021 20:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants