-
Notifications
You must be signed in to change notification settings - Fork 271
Adds a read the docs config file #1423
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
Conversation
how weird. a few issues: on the CI,
despite this PR not changing anything in the actual code base. also, the read the docs build failing despite me literally copying the files exactly from the successful #1419 setup |
Here's the failure traceback on rtd:
|
my bad - i mistakenly put the yaml in the docs/ directory rather than the root directory. |
LGTM |
* added attributes to mock player * fixed mock player addition * added test for size checking * added actions_size to Classifier * added actions_size to tutorial * re-added the actual implementation (git shenanigans) * fixed typo * fixed docstring * changed size checking to a general assumptions model * fixed some bugs * added some tests * added user option for strictness of assumption checking * fixed tests * updated docs * added assumptions_satisfy helper method * removed unnecessary if statement * Revert "removed unnecessary if statement" This reverts commit c4359bc. * added test for assumptions_satisfy * changed attributes -> characteristics * more attributes -> characteristics * changed name to game_characteristics when not directly referencing a Game * Adds a read the docs config file (#1423) * added a read the docs config file * moved yaml to root directory * removed default characteristic * made mock player safer
Fixes issues caused by readthedocs/readthedocs.org#10290. originally part of #1419.