Skip to content

Commit b70142d

Browse files
author
Chris Elion
authored
[MLA-1519] Don't mark action_probs as an output node. (#4613)
* remove action_probs from output nodes * changelog * pin cattrs upper version * print pip freeze results * add comment about cattrs version
1 parent 75f8a2b commit b70142d

File tree

4 files changed

+7
-5
lines changed

4 files changed

+7
-5
lines changed

.github/workflows/pytest.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,9 @@ jobs:
5151
python -m pip install --progress-bar=off -r test_requirements.txt -c ${{ matrix.pip_constraints }}
5252
python -m pip install --progress-bar=off -e ./gym-unity -c ${{ matrix.pip_constraints }}
5353
- name: Save python dependencies
54-
run: pip freeze > pip_versions-${{ matrix.python-version }}.txt
54+
run: |
55+
pip freeze > pip_versions-${{ matrix.python-version }}.txt
56+
cat pip_versions-${{ matrix.python-version }}.txt
5557
- name: Run pytest
5658
run: pytest --cov=ml-agents --cov=ml-agents-envs --cov=gym-unity --cov-report html --junitxml=junit/test-results-${{ matrix.python-version }}.xml -p no:warnings
5759
- name: Upload pytest test results

com.unity.ml-agents/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ and this project adheres to
2222
#### com.unity.ml-agents (C#)
2323
- The Barracuda dependency was upgraded to 1.1.2 (#4571)
2424
#### ml-agents / ml-agents-envs / gym-unity (Python)
25+
- The `action_probs` node is no longer listed as an output in TensorFlow models (#4613).
2526

2627
### Bug Fixes
2728
#### com.unity.ml-agents (C#)

ml-agents/mlagents/trainers/tf/model_serialization.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,7 @@
4040
]
4141
)
4242

43-
POSSIBLE_OUTPUT_NODES = frozenset(
44-
["action", "action_probs", "recurrent_out", "value_estimate"]
45-
)
43+
POSSIBLE_OUTPUT_NODES = frozenset(["action", "recurrent_out", "value_estimate"])
4644

4745
MODEL_CONSTANTS = frozenset(
4846
[

ml-agents/setup.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,8 @@ def run(self):
6767
# https://github.com/Unity-Technologies/ml-agents/blob/master/docs/Installation.md#windows-installing-pytorch
6868
'torch>=1.6.0,<1.8.0;platform_system!="Windows"',
6969
"tensorboard>=1.15",
70-
"cattrs>=1.0.0",
70+
# cattrs 1.1.0 dropped support for python 3.6.
71+
"cattrs>=1.0.0,<1.1.0",
7172
"attrs>=19.3.0",
7273
'pypiwin32==223;platform_system=="Windows"',
7374
],

0 commit comments

Comments
 (0)