-
Notifications
You must be signed in to change notification settings - Fork 7
Merge Dev/jan into experimental #4
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
Merged
Merged
Changes from all commits
Commits
Show all changes
78 commits
Select commit
Hold shift + click to select a range
45500ce
added log level command line arg
vonHartz 8f18596
testing poetry
vonHartz 1cdaa6a
playing with toml
vonHartz 628ab31
mutex problem
vonHartz ba13f35
fixed mutex extras
vonHartz d8a58c3
version bump for pypi test
vonHartz 730d2a0
added status badges
vonHartz d242a93
rebuild docs
vonHartz f74537b
rebuild docs
vonHartz c804624
added build docs to git for subtree use
vonHartz 9b09064
updated doc
vonHartz cdb34be
added nojekyll
vonHartz 0ccd74d
fixed command
vonHartz 9926e40
fixed cov link
vonHartz 1e1ad4a
added workflows from master
vonHartz 6fdc196
added missing build command
vonHartz 462df3d
bumped versions for deploy test, moved tests
vonHartz 183f803
fixed typo
vonHartz 4749c21
mujoco installation
vonHartz ed0bc10
debug mujoco install
vonHartz 3c861be
debug mujoco install
vonHartz 76571b9
added pypi isntall instructions, entry point (NOT FUNCTIONAL YET)
vonHartz 9b2837f
skip test, publish only when creating release
vonHartz f6f5e08
added author
vonHartz 6fe8ace
rm mujoco installation
vonHartz b9ad471
merged experimental into dev/jan
vonHartz 1663b8f
created main function
vonHartz 6b6cc90
ensured prev_reward is always defined
vonHartz 237c33f
another test commit to testpypi for testing installation via pip
vonHartz 793b744
reset build and release condition
vonHartz deb866c
renamed test workflow, added pypi installation test flow
vonHartz 012ea3c
removed ununsed packages"
vonHartz 115c6a5
ray installation fixed
vonHartz 4ddf94f
added entry point to pyproject for poetry
vonHartz bca0ccc
version bump, deploy for pypi test
vonHartz d085e44
reset trigger condition
vonHartz 77df7a2
typo
vonHartz c25e69b
generated lock file
vonHartz 22a4939
added init to find run_exp
vonHartz cd5cca4
reset trigger condition
vonHartz 030b78b
synced setup: updated urls, copied packages from poetry build
vonHartz 3ca5af3
moved scripts around to fix entry point
vonHartz 94ce46d
version bump for pypi test
vonHartz 4c475cd
reset trigger condition
vonHartz 070792c
added cov to worklfow, fixed test for new script dir
vonHartz a6cc098
allow to call main with args in other python scripts
vonHartz 19e18d8
moved scripts to top level, adapted setups
vonHartz 032ef4b
Merge branch 'experimental' into dev/jan
vonHartz 2d2104a
Revert "moved scripts to top level, adapted setups"
vonHartz 57e9d45
removed redundant import
vonHartz 651b5ac
reapplied cli change
vonHartz a312d22
reapplied cli entry point
vonHartz 55a0250
set version for pypi
vonHartz 47c8e68
updated workflows for release prp, fixed old arg, made comment more s…
vonHartz b4726ac
unified ray init that is version agnostic
vonHartz 48878f0
updated readmes
vonHartz 03ce4d6
removed other scripts for test
vonHartz 9ea98eb
deps defined in setup.py, so use this to install
vonHartz eafe368
need np for configspace
vonHartz d4c8aea
need np for configspace
vonHartz f52ac34
fixed gcc version, updated scipy version, added numpy to build reqs"
vonHartz e905008
install setuptools, not np
vonHartz caf519d
added setup requirement
vonHartz ad5373a
need manual numpy install
vonHartz 2e6f42e
moved to toml
vonHartz 0457d34
auto-install doesnt work
vonHartz 35977ca
added ray extras"
vonHartz 43a8516
used ray 0.9 for test due to beta_annealing_factor being depreceated
vonHartz 2740bd0
dont use mujoco in tests
vonHartz 6ec6be1
manual tf install, fixed tmp_dir arg for ray 0.x
vonHartz 611b815
updated authors
vonHartz d01b9d1
rebuild docs
vonHartz 24618cf
gym version
vonHartz 3e83971
ray versions for test
vonHartz 98f04cf
updated ray version
vonHartz 337e497
typo
vonHartz af56610
updated dep for test
vonHartz 400ac6e
install atari
vonHartz 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 hidden or 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,53 @@ | ||
# This workflow will install Python dependencies, run tests and lint with a single version of Python | ||
# For more information see: https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions | ||
|
||
name: Test | ||
|
||
on: | ||
push: | ||
branches: [ master, dev/jan ] | ||
pull_request: | ||
branches: [ master, dev/jan ] | ||
|
||
jobs: | ||
build: | ||
|
||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Set up Python 3.6 | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: 3.6 | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install flake8 pytest pytest-cov setuptools numpy | ||
pip install -e .[extras_disc] | ||
pip install ray[rllib,debug]==0.7.3 | ||
pip install tensorflow==1.13.0rc1 | ||
- name: Lint with flake8 | ||
run: | | ||
# stop the build if there are Python syntax errors or undefined names | ||
flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics | ||
# exit-zero treats all errors as warnings. | ||
flake8 . --count --exit-zero --statistics | ||
- name: Test with pytest | ||
run: | | ||
pytest tests | ||
- name: Cov report | ||
run: | | ||
pytest --cov=./ --cov-report=xml tests | ||
- name: Upload coverage to Codecov | ||
uses: codecov/codecov-action@v1 | ||
with: | ||
token: ${{ secrets.CODECOV_TOKEN }} | ||
files: ./coverage.xml | ||
directory: ./coverage/reports/ | ||
flags: unittests | ||
env_vars: OS,PYTHON | ||
name: codecov-umbrella | ||
fail_ci_if_error: true | ||
path_to_write_report: ./coverage/codecov_report.txt | ||
verbose: true |
This file contains hidden or 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,31 @@ | ||
name: Test PyPI Installation | ||
|
||
on: | ||
push: | ||
branches: [ master, dev/jan ] | ||
pull_request: | ||
branches: [ master, dev/jan ] | ||
|
||
jobs: | ||
build: | ||
|
||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Set up Python 3.6 | ||
uses: actions/setup-python@v2 | ||
with: | ||
python-version: 3.6 | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install tensorflow==2.2.0 | ||
wget 'https://ray-wheels.s3-us-west-2.amazonaws.com/master/8d0c1b5e068853bf748f72b1e60ec99d240932c6/ray-0.9.0.dev0-cp36-cp36m-manylinux1_x86_64.whl' | ||
pip install ray-0.9.0.dev0-cp36-cp36m-manylinux1_x86_64.whl[rllib,debug] | ||
- name: Install from PyPI | ||
run: | | ||
pip install -i https://test.pypi.org/pypi/ --extra-index-url https://pypi.org/simple mdp-playground | ||
- name: Basic functionality test | ||
run: | | ||
run-mdpp-experiments --help |
This file contains hidden or 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,40 @@ | ||
name: Build and Publish | ||
|
||
on: | ||
release: | ||
types: | ||
- created # see https://docs.github.com/en/rest/reference/repos#create-a-release and https://docs.github.com/en/actions/reference/events-that-trigger-workflows#release | ||
|
||
jobs: | ||
publish: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v2 | ||
- name: Set up Python | ||
uses: actions/setup-python@v1 | ||
with: | ||
python-version: "3.6" | ||
- name: Install Poetry | ||
run: | | ||
curl -fsS -o get-poetry.py https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | ||
python get-poetry.py -y | ||
echo "$HOME/.poetry/bin" >> $GITHUB_PATH | ||
- name: Configure poetry | ||
run: poetry config virtualenvs.in-project true | ||
- name: Set up cache | ||
uses: actions/cache@v1 | ||
id: cache | ||
with: | ||
path: .venv | ||
key: venv-${{ runner.os }}-${{ steps.full-python-version.outputs.version }}-${{ hashFiles('**/poetry.lock') }} | ||
- name: Ensure cache is healthy | ||
if: steps.cache.outputs.cache-hit == 'true' | ||
run: poetry run pip --version >/dev/null 2>&1 || rm -rf .venv | ||
- name: Install dependencies | ||
run: poetry install --no-interaction | ||
- name: Publish | ||
env: | ||
PYPI_TOKEN: ${{ secrets.PYPI_TOKEN }} | ||
run: | | ||
poetry config pypi-token.pypi $PYPI_TOKEN | ||
poetry publish --build -r pypi |
This file contains hidden or 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 |
---|---|---|
|
@@ -67,9 +67,6 @@ instance/ | |
# Scrapy stuff: | ||
.scrapy | ||
|
||
# Sphinx documentation | ||
docs/_build/ | ||
|
||
# PyBuilder | ||
target/ | ||
|
||
|
This file contains hidden or 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,44 @@ | ||
# Contributing | ||
|
||
## Developer Installation | ||
|
||
For development, manual installation is the easiest way to stay up-to-date: | ||
```bash | ||
pip install -e .[extras] | ||
``` | ||
|
||
In addition to the standard dependencies, please install the following: | ||
```bash | ||
pip install sphinx, sphinx-book-theme # for generating documentation | ||
pip install pytest-cov # for coverage report | ||
# install poetry for packaging and publishing to PyPI | ||
curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python - | ||
``` | ||
|
||
## Packaging | ||
|
||
The `pyproject.toml` is set up using `poetry`. | ||
|
||
Build via `poetry build`, and publish via `poetry publish`. There are automatic workflows in place to build and publish on new package revisions. | ||
|
||
To enable manual installation with `poetry`, we also include a `setup.py` which needs to be kept up-to-date. | ||
|
||
|
||
## Docs | ||
|
||
The documentation can be built using sphinx via: | ||
```bash | ||
cd docs | ||
make html | ||
``` | ||
|
||
To clean up: | ||
```bash | ||
make clean | ||
rm -rf _autosummary # optional | ||
``` | ||
|
||
To publish: | ||
```bash | ||
git subtree push --prefix docs/_build/html/ origin gh-pages | ||
``` |
This file contains hidden or 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
2 changes: 1 addition & 1 deletion
2
docs/_autosummary/mdp_playground.analysis.analysis.MDPP_Analysis.rst
This file contains hidden or 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
This file contains hidden or 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
73 changes: 0 additions & 73 deletions
73
docs/_autosummary/mdp_playground.analysis.radar_chart.Affine2D.rst
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.