Skip to content
This repository has been archived by the owner. It is now read-only.

Support gym without configure #162

Merged
merged 12 commits into from Mar 28, 2017
Merged

Support gym without configure #162

merged 12 commits into from Mar 28, 2017

Conversation

@tlbtlbtlb
Copy link
Contributor

@tlbtlbtlb tlbtlbtlb commented Mar 10, 2017

This works with gym 0.8.1

tlbtlbtlb added 8 commits Mar 10, 2017
Logger now takes print_frequency in constructor
@nottombrown
Copy link
Contributor

@nottombrown nottombrown commented Mar 12, 2017

Pushed a tweak to tox dependencies, that will hopefully fix travis

tlbtlbtlb added 3 commits Mar 16, 2017
@tlbtlbtlb tlbtlbtlb merged commit a4f78eb into master Mar 28, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
tlbtlbtlb added a commit that referenced this pull request May 15, 2017
* Major surgery to avoid gym.Env.configure

* track flashgames

* Forward configure through Timer-Render-Throttle

Logger now takes print_frequency in constructor

* More configure fixes. py.test passes locally

* Upgrade to gym 0.8

* Require 0.8.1

* configure bugfixes

* Record env metadata & spec. Use semantics.async instead of spec.vnc

* Bump tox version of gym for travis

* Fix race condition when recording from multiple threads on non-async envs

* sub-bump

* Bump
tlbtlbtlb added a commit that referenced this pull request May 18, 2017
* Set OPENAI_REMOTE_VERBOSE=0 to disable container logging

* Support gym without configure (#162)

* Major surgery to avoid gym.Env.configure

* track flashgames

* Forward configure through Timer-Render-Throttle

Logger now takes print_frequency in constructor

* More configure fixes. py.test passes locally

* Upgrade to gym 0.8

* Require 0.8.1

* configure bugfixes

* Record env metadata & spec. Use semantics.async instead of spec.vnc

* Bump tox version of gym for travis

* Fix race condition when recording from multiple threads on non-async envs

* sub-bump

* Bump

* #15 Update main readme to assist Windows users with getting started (#169)

* Install pytest in the docker image, and clean up and python cache files

* #15 Update main readme to assist Windows users with getting started

* #15 Update main readme to assist Windows users with getting started

* #15 Update main readme to assist Windows users with getting started

* #15 Update main readme to assist Windows users with getting started

* Fixes #171

Travis didn’t catch this. Perhaps we should test with `pip install` and
not  just copy the directory.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.