Skip to content
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

Paper branch #69

Merged
merged 42 commits into from Apr 25, 2022
Merged

Paper branch #69

merged 42 commits into from Apr 25, 2022

Conversation

conorheins
Copy link
Collaborator

  • JOSS manuscript now in paper folder
  • initial changes based on reviews from @seankmartin (added Summary section to paper, added documentation for Agent API)
  • other minor tweaks (some minor variable renaming in the internals of Agent)

@conorheins
Copy link
Collaborator Author

conorheins commented Mar 12, 2022

The new Agent API autodocumentation was in response to #67 by @seankmartin (see also openjournals/joss-reviews#4098 (comment)). That new documentation can be found on the RTD page specifically for the paper_branch, which can be found here: https://pymdp-rtd.readthedocs.io/en/paper_branch/agent.html

I am not going to close #67 because we still need to add API documentation for the Env base class to the RTD website to fully address @seankmartin's comments..

@conorheins
Copy link
Collaborator Author

conorheins commented Mar 13, 2022

In response to #67 and openjournals/joss-reviews#4098 (comment), we have now added a specific sub-section in the APIs documentation section that documents the Env base class and the sub-class implementations. That can be found here: https://pymdp-rtd.readthedocs.io/en/paper_branch/env.html

I think this now fully addresses #67

sphinx_rtd_theme
sphinx-autodoc-typehints==1.11.1
jupyter-sphinx>=0.3.2
myst-nb
jinja2==3.0.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see RDT issue here

…states and two instances of "THe" vs. "the"), to address Issue #71: #71 (comment)
Update paper_branch with new changes in the tutorial_scripts_translations branches (e.g. updates to the index.rst file) # the commit.
- added Verses affil for other co-authors, added Alec's 2nd sussex affiliation
…(comment)

- added missing DOIs for particular citations
- used official ForneyLab citation from their GitHub page
-standardized ArXiv-preprint reference style
…wledgement to be explicit about finding technical error in arxiv version
@conorheins conorheins merged commit 137cec4 into master Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant