chainer / chainerrl

ChainerRL is a deep reinforcement learning library built on top of Chainer.
MIT License
1.17k stars 224 forks source link

Fix CI errors due to pyglet, zipp, mock, and gym #592

Closed muupan closed 4 years ago

muupan commented 4 years ago
muupan commented 4 years ago

/test

pfn-ci-bot commented 4 years ago

Successfully created a job for commit 890f858:

muupan commented 4 years ago

/test

pfn-ci-bot commented 4 years ago

Successfully created a job for commit 65534f0:

toslunar commented 4 years ago

https://travis-ci.org/chainer/chainerrl/jobs/649363317#L2503

$ ./test_examples.sh -1
Running example tests: examples_tests/atlas/test_soft_actor_critic.sh
Output files are saved in /tmp/tmp.cdt7eztf9w/atlas/soft_actor_critic/20200212T100236.429911
/home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/gym/logger.py:30: UserWarning: WARN: Box bound precision lowered by casting to float32
  warnings.warn(colorize('%s: %s'%('WARN', msg % args), 'yellow'))
Traceback (most recent call last):
  File "examples/atlas/train_soft_actor_critic_atlas.py", line 243, in <module>
    main()
  File "examples/atlas/train_soft_actor_critic_atlas.py", line 123, in main
    timestep_limit = sample_env.spec.tags.get(
AttributeError: 'EnvSpec' object has no attribute 'tags'
The command "./test_examples.sh -1" exited with 1.

due to the version of gym?

pfn-ci-bot commented 4 years ago
  [INVALID_ARGUMENT] unknown command: home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/gym/logger.py:30: (686f6d652f7472617669732f7669727475616c656e762f707974686f6e332e362e372f6c69622f707974686f6e332e362f736974652d7061636b616765732f67796d2f6c6f676765722e70793a33303a)
  2020-02-12 19:19:58.399156 github_issue_comment.go:99] unknown command: home/travis/virtualenv/python3.6.7/lib/python3.6/site-packages/gym/logger.py:30: (686f6d652f7472617669732f7669727475616c656e762f707974686f6e332e362e372f6c69622f707974686f6e332e362f736974652d7061636b616765732f67796d2f6c6f676765722e70793a33303a)

  Stack trace:
    github.com/pfnet/imosci/util/frontend/handler/apihandler.(*githubWebhookIssueCommentFlow).Do (github_issue_comment.go:99)
    github.com/pfnet/imosci/util/frontend/handler/apihandler.githubIssueCommentHandler (github_issue_comment.go:45)
    runtime.call64 (asm_amd64.s:523)
    reflect.Value.call (value.go:447)
    reflect.Value.Call (value.go:308)
    github.com/pfnet/imosci/util/frontend/core.RegisterAPIHandlerInternal.func1 (handler.go:417)
    github.com/pfnet/imosci/util/frontend/core.RegisterHandler.func1 (handler.go:173)
    github.com/pfnet/imosci/util/frontend/core.RegisterHandler.func2.1 (handler.go:275)
    github.com/pfnet/imosci/util/frontend/core.RegisterHandler.func2 (handler.go:280)
    net/http.HandlerFunc.ServeHTTP (server.go:1964)
    net/http.(*ServeMux).ServeHTTP (server.go:2361)
    github.com/pfnet/imosci/util/api.callInternal.func2 (call.go:196)
    github.com/pfnet/imosci/util/api.callInternal (call.go:204)
    github.com/pfnet/imosci/util/api.Call (call.go:120)
    github.com/pfnet/imosci/util/api.GithubIssueComment (call.go:492)
    github.com/pfnet/imosci/util/frontend/handler/xternalhandler.githubWebhookHandler (github_webhook.go:118)
    github.com/pfnet/imosci/util/frontend/core.RegisterHandler.func1 (handler.go:173)
    github.com/pfnet/imosci/util/frontend/core.RegisterHandler.func2.1 (handler.go:275)
    github.com/pfnet/imosci/util/frontend/core.RegisterHandler.func2 (handler.go:280)
    net/http.HandlerFunc.ServeHTTP (server.go:1964)
    net/http.(*ServeMux).ServeHTTP (server.go:2361)
    google.golang.org/appengine/internal.executeRequestSafely (api.go:162)
    google.golang.org/appengine/internal.handleHTTP (api.go:121)
    net/http.HandlerFunc.ServeHTTP (server.go:1964)
    net/http.serverHandler.ServeHTTP (server.go:2741)
    net/http.(*conn).serve (server.go:1847)
    runtime.goexit (asm_amd64.s:1333)
muupan commented 4 years ago

/test

pfn-ci-bot commented 4 years ago

Successfully created a job for commit 73a1a44:

muupan commented 4 years ago

/test

pfn-ci-bot commented 4 years ago

Successfully created a job for commit 7fb7212:

muupan commented 4 years ago

/test

pfn-ci-bot commented 4 years ago

Successfully created a job for commit 54b49c9:

muupan commented 4 years ago

@toslunar I added changes to fix errors caused by gym==0.16.0. Now CI is all ok. Can you check the additional changes and merge if it's ok?

toslunar commented 4 years ago

I confirmed env.spec.max_episode_steps is available with gym 0.9.7, the minimum requirement by ChainerRL.