-
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
Develop package docs #3551
Develop package docs #3551
Conversation
…he package docs page.
To use this package, you must have the following 3rd party products: | ||
|
||
* <TBC> |
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.
We could link to our python install docs here
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.
Is another Unity (Python) package considered a third-party?
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.
No, maybe we need a different section though to call that out as a requirement for training.
# About Unity MLAgents package |
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.
according to the Package Starter Kit I think this file is supposed to be named com.unity.ml-agents.md
.
Note that this package does not contain the machine learning algorithms for training | ||
behaviors. It relies on a Python package to orchestrate the training. This package | ||
only enables instrumenting a Unity scene and setting it up for training. |
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.
Do we want to reference inference via the Barracuda package?
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.
Good call. @surfnerd - is there a way to link to other packages here? We can just reference them by name too.
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.
We can link to their docs on the unity site.
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.
If you are a researcher interested in a discussion of Unity as an AI platform, see a pre-print | ||
of our [reference paper on Unity and the ML-Agents Toolkit](https://arxiv.org/abs/1809.02627). | ||
|
||
If you use Unity or the ML-Agents Toolkit to conduct research, we ask that you cite the following |
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.
how about we "request" instead of asking them to cite?
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.
Not sure I follow...
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 meant that we use
"... conduct research, we request that you cite the following ..."
instead of
"... conduct research, we ask that you cite the following ... "
Proposed change(s)
Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)
Types of change(s)
Checklist
Other comments