Break pcs out into separate dir and fix some broken example_agents #328
Conversation
…roject#329 but not sure there are other tests besides the 2 that I updated in this commit that need to clear the local repo cache as well.
…django webapp, add local repo cache clearing to webapp tests which further addresses agentos-project#329.
…the deletion fails becuase modules from those repos have been loaded into the current Python process.
example rl_agents.
|
@nickjalbert I think this and #326 are ready for review! |
andyk
changed the title
nickjalbert
left a comment
nickjalbert
left a comment
There was a problem hiding this comment.
LGTM except it appears python scripts/build_docs.py is broken (it's still trying to import agentos/version.py instead of pcs/version.py). We'll also have to update the autodoc stubs in documentation/api to point to the new pcs location.
Not sure if that's what you meant by the docs not being updated, but we could port that in a different PR if you want to merge this one now.
for running these agents.
|
Ok @nickjalbert I think I've addressed all of your feedback and updated the main text of this PR to reflect the updates. If tests are green I think we are in good shape, though might be nice if you try out the newly updated |
andyk
changed the title
|
Actually I just noticed this PR still doesn't add readme files to all of the example agents, which is one of the things you suggested. |
|
Build is green! Merging! Thanks for doing this. :) |
2 participants
Fixes #289 and #337
This updates the API docs and also updates the
build_docs.pyscript to auto-generate the API docs with less manual work required to curate stub sphinx ReStructuredText files.This also fixes some syntax bugs in the example code in the REPL Quickstart in the documentation.