-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[MLA-1912] Move Match3 to com.unity.ml-agents #5259
Conversation
@@ -18,8 +18,7 @@ The following table describes the package folder structure: | |||
| _Runtime_ | Contains core C# APIs for integrating ML-Agents into your Unity scene. | | |||
| _Tests_ | Contains the unit tests for the package. | | |||
|
|||
The Runtime directory currently contains three features: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The count was wrong before :) It would be correct now, but made it more generic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made the exact same change in my PR :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Proposed change(s)
Move the match3 integration and update the docs.
Note I need to move the public sensor and actuator components to private, add properties, and add custom editors. I'll do that in a later PR, just want to get the move done now.
Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)
Types of change(s)
Checklist
Other comments