forked from openai/baselines
-
Notifications
You must be signed in to change notification settings - Fork 724
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
Add Windows CI #674
Open
ChengYen-Tang
wants to merge
17
commits into
hill-a:master
Choose a base branch
from
NTUT-SELab:WindowsCI
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+135
−17
Open
Add Windows CI #674
Changes from 7 commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
0f09683
pytype not support windows.
ChengYen-Tang f106ba5
Test Install MSMPI
ChengYen-Tang 284b859
Test Install Package (+2 squashed commit)
ChengYen-Tang 0c907d3
UnitTests_a
ChengYen-Tang 176e334
UnitTests_a_h
ChengYen-Tang 1b1b091
UnitTests
ChengYen-Tang 748335d
test build ale_c
ChengYen-Tang df66702
Updated the changelog
ChengYen-Tang ef1c821
Fix test_monitor() failed
ChengYen-Tang 8fc25e7
Use cmd.exe run test
ChengYen-Tang c8b1386
Listing Windows Installer Applications.
ChengYen-Tang ad3754a
The process cannot access the file because it is being used by anothe…
ChengYen-Tang 2d6cd7f
Merge branch 'pr/24' into WindowsCI
ChengYen-Tang 14c3597
Test_mpi_adam
ChengYen-Tang 042f6e4
update job name
ChengYen-Tang 1006a7e
Update MPI command.
ChengYen-Tang 502ac35
Merge branch 'WindowsCI' of https://github.com/NTUT-SELab/stable-base…
ChengYen-Tang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,47 @@ | ||
pool: | ||
vmImage: 'windows-latest' | ||
|
||
variables: | ||
PYTHON_VERSION: 3.6 | ||
|
||
jobs: | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_a_d' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '[a-d]*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_e_h' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '[e-h]*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_i_lo' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '[i-lo]*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_lp_lz' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '[lp-lz]*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_m_r' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '[m-r]*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_s' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: 's*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_t_z' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '[t-z]*' | ||
- template: azure_templates/run-test.yml | ||
parameters: | ||
name: 'UnitTests_determinism' | ||
PYTHON_VERSION: $(PYTHON_VERSION) | ||
GLOB: '0deterministic.py' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
steps: | ||
- task: PowerShell@2 | ||
inputs: | ||
targetType: 'inline' | ||
script: | | ||
Invoke-WebRequest "https://github.com/microsoft/Microsoft-MPI/releases/download/v10.1.1/msmpisetup.exe" -OutFile $pwd\msmpisetup.exe ; | ||
Start-Process $pwd\msmpisetup.exe -ArgumentList '-unattend' -Wait ; | ||
Remove-Item $pwd\msmpisetup.exe -Force |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
steps: | ||
- script: | | ||
python -m pip install --upgrade pip --user | ||
python -m pip install --upgrade setuptools | ||
python -m pip install tensorflow==1.14.0 | ||
python -m pip install -e .[mpi,tests,docs] | ||
displayName: 'Install python package' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
jobs: | ||
- job: ${{ parameters.name }} | ||
steps: | ||
- task: UsePythonVersion@0 | ||
inputs: | ||
versionSpec: ${{ parameters.PYTHON_VERSION }} | ||
- template: install-msmpi.yml | ||
- template: install-python-package.yml | ||
- script: bash -c "python -m pytest --cov-config .coveragerc --cov-report term --cov=. -v tests/test_${{ parameters.GLOB }}" | ||
displayName: 'Run test' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,7 @@ | ||
import os | ||
import sys | ||
import subprocess | ||
import platform | ||
from setuptools import setup, find_packages | ||
from distutils.version import LooseVersion | ||
|
||
|
@@ -39,6 +40,13 @@ def find_tf_dependency(): | |
|
||
return tf_dependency | ||
|
||
def other_tests_packages(): | ||
if platform.system() == 'Windows': | ||
return [] | ||
elif platform.system() == 'Darwin': # Mac OS X | ||
return ['pytype'] | ||
else: | ||
return ['pytype'] | ||
|
||
Comment on lines
42
to
50
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Pytype is currently not support windows. |
||
long_description = """ | ||
[![Build Status](https://travis-ci.com/hill-a/stable-baselines.svg?branch=master)](https://travis-ci.com/hill-a/stable-baselines) [![Documentation Status](https://readthedocs.org/projects/stable-baselines/badge/?version=master)](https://stable-baselines.readthedocs.io/en/master/?badge=master) [![Codacy Badge](https://api.codacy.com/project/badge/Grade/3bcb4cd6d76a4270acb16b5fe6dd9efa)](https://www.codacy.com/app/baselines_janitors/stable-baselines?utm_source=github.com&utm_medium=referral&utm_content=hill-a/stable-baselines&utm_campaign=Badge_Grade) [![Codacy Badge](https://api.codacy.com/project/badge/Coverage/3bcb4cd6d76a4270acb16b5fe6dd9efa)](https://www.codacy.com/app/baselines_janitors/stable-baselines?utm_source=github.com&utm_medium=referral&utm_content=hill-a/stable-baselines&utm_campaign=Badge_Coverage) | ||
|
@@ -135,8 +143,7 @@ def find_tf_dependency(): | |
'pytest-cov', | ||
'pytest-env', | ||
'pytest-xdist', | ||
'pytype', | ||
], | ||
] + other_tests_packages(), | ||
'docs': [ | ||
'sphinx', | ||
'sphinx-autobuild', | ||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
On Windows, the python version must be> = 3.6 to install atari-py.