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
[refactor] Add --tensorflow, enable Torch as default setting #4582
Conversation
@@ -169,10 +169,10 @@ def _create_parser() -> argparse.ArgumentParser: | |||
help="Forces training using CPU only", | |||
) | |||
argparser.add_argument( |
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 think we should still have --torch but raise a warning saying it is useless and it will be removed in a future release.
Similarly, we should raise a warning if the user is using --tensorflow as it will be deprecated in a future release.
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.
Added the warnings and --torch back
* Torch setup.py * Set torch to default * Make torch default in setup.py * Remove indents * Remove other instances of TF being used * Add tensorboard to setup.py * Adding correst setup commands for verifying torch is installed (#4524) * Adding correst setup commands for verifying torch is installed * Editing the test_requirments to add tf and remove torch * Develop torchdefault raise outside setup (#4530) * Torch not imported error to raise at first usage * Torch not imported error to raise at first usage * [refactor] Use PyTorch TensorBoard utils (#4518) * Convert stats writer to use PyTorch TB support * Use common function to print params * Update test * Bump tensorboard to 1.15 to fix the tests * putting tensorboard 1.15.0 as min version requirement Co-authored-by: vincentpierre <vincentpierre@unity3d.com> * [Docs] Initial documentation changes for making Torch the default (#4561) * Initial commit * Forgotten doc * Removing the `Installation-Anaconda-Windows.md` as it is deprecated * Readding the depreacted Installation-Anaconda-Windows.md but leaving it unchanged * more references to tensorflow removed * Update README.md Co-authored-by: Ervin T. <ervin@unity3d.com> * Change references to .nn to .onnx in docs (#4583) Co-authored-by: Ervin T. <ervin@unity3d.com> * [refactor] Add --tensorflow, enable Torch as default setting (#4582) * Add --tensorflow option * Switch framework to Pytorch default * Update changelog * Re-add --torch * Edit warning * Modify Yamato tests (#4584) * Don't check for PB file in Yamato inference * Only run inference on ONNX * Update docs/Unity-Inference-Engine.md with correct tf2onnx versions Co-authored-by: Chris Elion <chris.elion@unity3d.com> * Add reward signal class comments * More descriptive import of is_available * Updated installation instructions for PyTorch * More reward signal comments * More Windows instructions Co-authored-by: Vincent-Pierre BERGES <vincentpierre@unity3d.com> Co-authored-by: Chris Elion <chris.elion@unity3d.com>
Proposed change(s)
Add a
--tensorflow
option to revert back to TensorFlow.Also actually changes the default framework in settings to PyTorch, and adds a changelog entry.
Note:
pip install mlagents[tensorflow]
was already added in the branch.Types of change(s)
Checklist
Other comments