openjournals / joss-reviews

Reviews for the Journal of Open Source Software
Creative Commons Zero v1.0 Universal
725 stars 38 forks source link

[REVIEW]: pymdp: A Python library for active inference in discrete state spaces #4098

Closed whedon closed 2 years ago

whedon commented 2 years ago

Submitting author: !--author-handle-->@conorheins<!--end-author-handle-- (Conor Heins) Repository: https://github.com/infer-actively/pymdp Branch with paper.md (empty if default branch): master Version: v0.0.5 Editor: !--editor-->@emdupre<!--end-editor-- Reviewers: @seankmartin, @patrickmineault Archive: 10.5281/zenodo.6484849

:warning: JOSS reduced service mode :warning:

Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.

Status

status

Status badge code:

HTML: <a href="https://joss.theoj.org/papers/a6362c8d3e0c953d8ce832664a32f66b"><img src="https://joss.theoj.org/papers/a6362c8d3e0c953d8ce832664a32f66b/status.svg"></a>
Markdown: [![status](https://joss.theoj.org/papers/a6362c8d3e0c953d8ce832664a32f66b/status.svg)](https://joss.theoj.org/papers/a6362c8d3e0c953d8ce832664a32f66b)

Reviewers and authors:

Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)

Reviewer instructions & questions

@seankmartin & @patrickmineault, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:

  1. Make sure you're logged in to your GitHub account
  2. Be sure to accept the invite at this URL: https://github.com/openjournals/joss-reviews/invitations

The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @emdupre know.

Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest

Review checklist for @seankmartin

✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨

Conflict of interest

Code of Conduct

General checks

Functionality

Documentation

Software paper

Review checklist for @patrickmineault

✨ Important: Please do not use the Convert to issue functionality when working through this checklist, instead, please open any new issues associated with your review in the software repository associated with the submission. ✨

Conflict of interest

Code of Conduct

General checks

Functionality

Documentation

Software paper

emdupre commented 2 years ago

Hi @conorheins ,

Thank you for your patience on this ! I've consulted with other JOSS editors and have outlined a list of next steps that I hope will be helpful. I'm detailing them below, and please do let me know if you have any questions in addressing them.

Pending these changes, could you then please:

I can then move forward with accepting the submission 🚀

conorheins commented 2 years ago

Hi @emdupre , I've now finished the re-formatting steps and changes to Statement of Need that you suggested.

Before I move onto the second set of steps (e.g. archiving the reviewed software in Zenodo, ... .), I just had a clarifying question about the order in which I do these steps.

The first step is "Make a tagged release of your software, and list the version tag of the archived version here".

Does this "archived version" refer to the archived version that will be generated in the subsequent steps (with Zenodo, etc.), or something else that I'm not understanding? This is probably obvious for someone who has worked with Zenodo or similar services but this is new for me, so before proceeding I just want to know what exactly I should be listing in the version tag of the tagged release.

emdupre commented 2 years ago

@editorialbot generate pdf

editorialbot commented 2 years ago

:warning: An error happened when generating the pdf.

emdupre commented 2 years ago

Hi @conorheins,

Thank you for addressing these points ! There's a typo here that's preventing PDF compilation. If you can remove the extra } character, we should be able to re-generate the PDF !

Does this "archived version" refer to the archived version that will be generated in the subsequent steps (with Zenodo, etc.), or something else that I'm not understanding? This is probably obvious for someone who has worked with Zenodo or similar services but this is new for me, so before proceeding I just want to know what exactly I should be listing in the version tag of the tagged release.

This is a good question; thank you for confirming ! What I mean is that when you create a new release of your software, this release is the one that is then archived in subsequent steps (e.g. on Zenodo using this workflow), as you suggested. The motivation is that you have a tagged release that includes the state of the software as it is described in the JOSS paper, and by archiving this tagged release it will have a persistent identifier; e.g. a DOI, that we can include in the JOSS publication.

Hopefully that helps to answer your questions, but please let me know if there's anything else I can clarify !

conorheins commented 2 years ago

Hi @emdupre, thanks for the clarification.

I have fixed the typo in the .bib file that should address this: https://github.com/openjournals/joss-reviews/issues/4098#issuecomment-1108998473

As requested, here is a link to the latest release: v0.0.5

And here is the DOI (from Zenodo): 10.5281/zenodo.6484849

I have also verified that the Zenodo archive version has the correct metadata (title and author list).

emdupre commented 2 years ago

@editorialbot generate pdf

editorialbot commented 2 years ago

I'm sorry human, I don't understand that. You can see what commands I support by typing:

@editorialbot commands

emdupre commented 2 years ago

@editorialbot generate pdf

editorialbot commented 2 years ago

:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:

emdupre commented 2 years ago

@editorialbot set version v0.0.5

editorialbot commented 2 years ago

I'm sorry human, I don't understand that. You can see what commands I support by typing:

@editorialbot commands

emdupre commented 2 years ago

@editorialbot set v0.0.5 as version

editorialbot commented 2 years ago

Done! version is now v0.0.5

emdupre commented 2 years ago

@editorialbot check references

emdupre commented 2 years ago

@editorialbot set 10.5281/zenodo.6484849 as archive

editorialbot commented 2 years ago

Done! Archive is now 10.5281/zenodo.6484849

emdupre commented 2 years ago

@editorialbot check references

editorialbot commented 2 years ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1371/journal.pone.0006421 is OK
- 10.1162/neco_a_01427 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1162/neco_a_01354 is OK
- 10.1080/17588928.2015.1020053 is OK
- 10.1038/s41598-018-38246-3 is OK
- 10.1561/2200000001 is OK
- 10.1167/jov.20.4.12 is OK
- 10.1038/ncomms14218 is OK
- 10.3390/e22050552 is OK
- 10.1162/neco.1992.4.3.415 is OK
- 10.1016/j.neuroimage.2004.07.041 is OK
- 10.1080/01621459.2017.1285773 is OK
- 10.1126/science.1238406 is OK
- 10.1007/s00422-012-0512-8 is OK
- 10.1016/j.tics.2011.11.018 is OK
- 10.1162/neco_a_01357 is OK
- 10.1109/IJCNN48605.2020.9207382 is OK
- 10.1016/0166-2236(83)90190-X is OK
- 10.1523/JNEUROSCI.11-03-00641.1991 is OK
- 10.1162/NECO_a_00912 is OK
- 10.3390/e21030257 is OK
- 10.3389/frobt.2019.00020 is OK
- 10.1007/s00422-018-0785-7 is OK
- 10.1371/journal.pcbi.1007805 is OK
- 10.1109/ICASSP40776.2020.9054364 is OK
- 10.1016/B978-0-12-372560-8.X5000-1 is OK
- 10.1038/nrn2787 is OK
- 10.1016/j.jmp.2020.102447 is OK
- 10.1016/S0004-3702(98)00023-X is OK
- 10.1162/neco_a_01351 is OK
- 10.1093/cercor/bhu159 is OK
- 10.1016/j.drugalcdep.2020.108208 is OK
- 10.1503/jpn.200032 is OK
- 10.1016/j.jmp.2021.102632 is OK
- 10.1016/j.neuroimage.2008.02.054 is OK
- 10.1016/j.jmp.2020.102348 is OK
- 10.1007/978-3-030-64919-7_1 is OK
- 10.1016/j.ijar.2018.11.002 is OK
- 10.1098/rstb.1980.0090 is OK
- 10.1016/j.neubiorev.2016.06.022 is OK
- 10.1162/neco.1995.7.5.889 is OK
- 10.1037/13536-000 is OK
- 10.1016/j.neunet.2021.05.010 is OK
- 10.1109/LRA.2021.3090015 is OK
- 10.1016/B978-0-323-88506-5.50066-8 is OK
- 10.1007/s00422-019-00805-w is OK
- 10.3390/e23020198 is OK
- 10.1080/10407413.2021.1965480 is OK
- 10.1098/rsif.2013.0475 is OK
- 10.7554/eLife.41703 is OK
- 10.1162/neco_a_00999 is OK
- 10.1016/j.neubiorev.2021.09.038 is OK
- 10.1016/j.schres.2021.07.032 is OK
- 10.1038/s41598-021-91308-x is OK

MISSING DOIs

- 10.1017/s0140525x19001353 may be a valid DOI for title: Generative models as parsimonious descriptions of sensorimotor loops
- 10.1007/978-3-030-93736-2_48 may be a valid DOI for title: Towards stochastic fault-tolerant control using precision learning and active inference

INVALID DOIs

- None
emdupre commented 2 years ago

Thank you, @conorheins !

A few remaining points:

These first two edits can be made directly on Zenodo by editing the metadata associated with the archive, as described here !

conorheins commented 2 years ago

Hi @emdupre,

Re: software archive metadata. Sorry about that, that's strange. I changed those metadata (title / author list) to the correct versions after I made the Zenodo release, but it seems it doesn't show up for the public-facing version, presumably because I didn't click "Publish" yet, only "Save". I've now clicked "Publish" so all the metadata should be consistent now. Let me know if you still notice any discrepancies. The title and author list should be identical to those on the paper now, hopefully.

re: Baltieri referrences -- baltieri2019pid (an entropy paper) is actually a different paper than baltieri2019 (a commentary on another paper), so I think that should be fine.

re: baouimy2021towards -- you are correct that I intended to keep the arXiv version listed.

emdupre commented 2 years ago

Great, thank you ! Only the @baltieri2019pid citation is currently included in the paper.md, so I wanted to confirm their relationship. I think, though, that having an uncited reference in the .bib file is not against JOSS policy, so we can move forward as-is.

I'm now happy to recommend pymdp to the EIC team for publication, and I just want to add my congratulations to you on such an impressive effort !

emdupre commented 2 years ago

@editorialbot recommend-accept

editorialbot commented 2 years ago
Attempting dry run of processing paper acceptance...
editorialbot commented 2 years ago

:wave: @openjournals/joss-eics, this paper is ready to be accepted and published.

Check final proof :point_right: https://github.com/openjournals/joss-papers/pull/3181

If the paper PDF and the deposit XML files look good in https://github.com/openjournals/joss-papers/pull/3181, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept

conorheins commented 2 years ago

Hi @emdupre,

Awesome, thank you! This is great news.

Before the paper is published, would now be a good time for me to just pass it around the co-authors to make sure everyone's name / affiliations are listed properly? Or is there an equivalent of a 'final proofs' stage coming later, at which small edits like that would be appropriate?

Thanks, Conor

emdupre commented 2 years ago

Please pass it around now ! This is effectively the "final proofs" stage, so we can still incorporate these corrections as well as any others that the EiCs suggest.

Please let us know when this is confirmed on your side !

conorheins commented 2 years ago

Ok sounds good, will do!

editorialbot commented 2 years ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1371/journal.pone.0006421 is OK
- 10.1162/neco_a_01427 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1162/neco_a_01354 is OK
- 10.1080/17588928.2015.1020053 is OK
- 10.1038/s41598-018-38246-3 is OK
- 10.1561/2200000001 is OK
- 10.1167/jov.20.4.12 is OK
- 10.1038/ncomms14218 is OK
- 10.3390/e22050552 is OK
- 10.1162/neco.1992.4.3.415 is OK
- 10.1016/j.neuroimage.2004.07.041 is OK
- 10.1080/01621459.2017.1285773 is OK
- 10.1126/science.1238406 is OK
- 10.1007/s00422-012-0512-8 is OK
- 10.1016/j.tics.2011.11.018 is OK
- 10.1162/neco_a_01357 is OK
- 10.1109/IJCNN48605.2020.9207382 is OK
- 10.1016/0166-2236(83)90190-X is OK
- 10.1523/JNEUROSCI.11-03-00641.1991 is OK
- 10.1162/NECO_a_00912 is OK
- 10.3390/e21030257 is OK
- 10.3389/frobt.2019.00020 is OK
- 10.1007/s00422-018-0785-7 is OK
- 10.1371/journal.pcbi.1007805 is OK
- 10.1109/ICASSP40776.2020.9054364 is OK
- 10.1016/B978-0-12-372560-8.X5000-1 is OK
- 10.1038/nrn2787 is OK
- 10.1016/j.jmp.2020.102447 is OK
- 10.1016/S0004-3702(98)00023-X is OK
- 10.1162/neco_a_01351 is OK
- 10.1093/cercor/bhu159 is OK
- 10.1016/j.drugalcdep.2020.108208 is OK
- 10.1503/jpn.200032 is OK
- 10.1016/j.jmp.2021.102632 is OK
- 10.1016/j.neuroimage.2008.02.054 is OK
- 10.1016/j.jmp.2020.102348 is OK
- 10.1007/978-3-030-64919-7_1 is OK
- 10.1016/j.ijar.2018.11.002 is OK
- 10.1098/rstb.1980.0090 is OK
- 10.1016/j.neubiorev.2016.06.022 is OK
- 10.1162/neco.1995.7.5.889 is OK
- 10.1037/13536-000 is OK
- 10.1016/j.neunet.2021.05.010 is OK
- 10.1109/LRA.2021.3090015 is OK
- 10.1016/B978-0-323-88506-5.50066-8 is OK
- 10.1007/s00422-019-00805-w is OK
- 10.3390/e23020198 is OK
- 10.1080/10407413.2021.1965480 is OK
- 10.1098/rsif.2013.0475 is OK
- 10.7554/eLife.41703 is OK
- 10.1162/neco_a_00999 is OK
- 10.1016/j.neubiorev.2021.09.038 is OK
- 10.1016/j.schres.2021.07.032 is OK
- 10.1038/s41598-021-91308-x is OK

MISSING DOIs

- Errored finding suggestions for Variational Algorithms for Approximate Bayesian Inference, please try later
- Errored finding suggestions for Reinforcement learning through active inference, please try later
- 10.1017/s0140525x19001353 may be a valid DOI for title: Generative models as parsimonious descriptions of sensorimotor loops
- 10.1007/978-3-030-93736-2_48 may be a valid DOI for title: Towards stochastic fault-tolerant control using precision learning and active inference

INVALID DOIs

- None
emdupre commented 2 years ago

When you're updating affiliations, @conorheins, can you please also:

I received additional guidance that we will need the the version-of-record in the JOSS publication.

conorheins commented 2 years ago

Hi @emdupre, I've gotten confirmation that affiliations/names are all correct :) I've also updated .bib file as requested

emdupre commented 2 years ago

@editorialbot check references

editorialbot commented 2 years ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1371/journal.pone.0006421 is OK
- 10.1162/neco_a_01427 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1162/neco_a_01354 is OK
- 10.1080/17588928.2015.1020053 is OK
- 10.1038/s41598-018-38246-3 is OK
- 10.1561/2200000001 is OK
- 10.1167/jov.20.4.12 is OK
- 10.1038/ncomms14218 is OK
- 10.3390/e22050552 is OK
- 10.1162/neco.1992.4.3.415 is OK
- 10.1016/j.neuroimage.2004.07.041 is OK
- 10.1080/01621459.2017.1285773 is OK
- 10.1126/science.1238406 is OK
- 10.1007/s00422-012-0512-8 is OK
- 10.1016/j.tics.2011.11.018 is OK
- 10.1162/neco_a_01357 is OK
- 10.1109/IJCNN48605.2020.9207382 is OK
- 10.1016/0166-2236(83)90190-X is OK
- 10.1523/JNEUROSCI.11-03-00641.1991 is OK
- 10.1162/NECO_a_00912 is OK
- 10.3390/e21030257 is OK
- 10.3389/frobt.2019.00020 is OK
- 10.1007/s00422-018-0785-7 is OK
- 10.1371/journal.pcbi.1007805 is OK
- 10.1109/ICASSP40776.2020.9054364 is OK
- 10.1016/B978-0-12-372560-8.X5000-1 is OK
- 10.1038/nrn2787 is OK
- 10.1016/j.jmp.2020.102447 is OK
- 10.1016/S0004-3702(98)00023-X is OK
- 10.1162/neco_a_01351 is OK
- 10.1093/cercor/bhu159 is OK
- 10.1016/j.drugalcdep.2020.108208 is OK
- 10.1503/jpn.200032 is OK
- 10.1016/j.jmp.2021.102632 is OK
- 10.1016/j.neuroimage.2008.02.054 is OK
- 10.1016/j.jmp.2020.102348 is OK
- 10.1007/978-3-030-64919-7_1 is OK
- 10.1016/j.ijar.2018.11.002 is OK
- 10.1098/rstb.1980.0090 is OK
- 10.1016/j.neubiorev.2016.06.022 is OK
- 10.1162/neco.1995.7.5.889 is OK
- 10.1037/13536-000 is OK
- 10.1016/j.neunet.2021.05.010 is OK
- 10.1109/LRA.2021.3090015 is OK
- 10.1016/B978-0-323-88506-5.50066-8 is OK
- 10.1007/s00422-019-00805-w is OK
- 10.3390/e23020198 is OK
- 10.1080/10407413.2021.1965480 is OK
- 10.1098/rsif.2013.0475 is OK
- 10.7554/eLife.41703 is OK
- 10.1162/neco_a_00999 is OK
- 10.1016/j.neubiorev.2021.09.038 is OK
- 10.1016/j.schres.2021.07.032 is OK
- 10.1038/s41598-021-91308-x is OK

MISSING DOIs

- 10.1017/s0140525x19001353 may be a valid DOI for title: Generative models as parsimonious descriptions of sensorimotor loops
- 10.1007/978-3-030-93736-2_48 may be a valid DOI for title: Towards stochastic fault-tolerant control using precision learning and active inference
- 10.3389/fpsyg.2022.855074 may be a valid DOI for title: Enactive-Dynamic Social Cognition and Active Inference

INVALID DOIs

- None
emdupre commented 2 years ago

@editorialbot set master as branch

editorialbot commented 2 years ago

Done! branch is now master

emdupre commented 2 years ago

@editorialbot check references

editorialbot commented 2 years ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1371/journal.pone.0006421 is OK
- 10.1162/neco_a_01427 is OK
- 10.1038/s41586-020-2649-2 is OK
- 10.1162/neco_a_01354 is OK
- 10.1080/17588928.2015.1020053 is OK
- 10.1038/s41598-018-38246-3 is OK
- 10.1561/2200000001 is OK
- 10.1167/jov.20.4.12 is OK
- 10.1038/ncomms14218 is OK
- 10.3390/e22050552 is OK
- 10.1162/neco.1992.4.3.415 is OK
- 10.1016/j.neuroimage.2004.07.041 is OK
- 10.1080/01621459.2017.1285773 is OK
- 10.1126/science.1238406 is OK
- 10.1007/s00422-012-0512-8 is OK
- 10.1016/j.tics.2011.11.018 is OK
- 10.1162/neco_a_01357 is OK
- 10.1109/IJCNN48605.2020.9207382 is OK
- 10.1016/0166-2236(83)90190-X is OK
- 10.1523/JNEUROSCI.11-03-00641.1991 is OK
- 10.1162/NECO_a_00912 is OK
- 10.3390/e21030257 is OK
- 10.3389/frobt.2019.00020 is OK
- 10.1007/s00422-018-0785-7 is OK
- 10.1371/journal.pcbi.1007805 is OK
- 10.1109/ICASSP40776.2020.9054364 is OK
- 10.1016/B978-0-12-372560-8.X5000-1 is OK
- 10.1038/nrn2787 is OK
- 10.1016/j.jmp.2020.102447 is OK
- 10.1016/S0004-3702(98)00023-X is OK
- 10.1162/neco_a_01351 is OK
- 10.1093/cercor/bhu159 is OK
- 10.1016/j.drugalcdep.2020.108208 is OK
- 10.1503/jpn.200032 is OK
- 10.1016/j.jmp.2021.102632 is OK
- 10.1016/j.neuroimage.2008.02.054 is OK
- 10.1016/j.jmp.2020.102348 is OK
- 10.1007/978-3-030-64919-7_1 is OK
- 10.1016/j.ijar.2018.11.002 is OK
- 10.1098/rstb.1980.0090 is OK
- 10.1016/j.neubiorev.2016.06.022 is OK
- 10.1162/neco.1995.7.5.889 is OK
- 10.1037/13536-000 is OK
- 10.1007/978-3-030-93736-2_48 is OK
- 10.1016/j.neunet.2021.05.010 is OK
- 10.1109/LRA.2021.3090015 is OK
- 10.1016/B978-0-323-88506-5.50066-8 is OK
- 10.1007/s00422-019-00805-w is OK
- 10.3390/e23020198 is OK
- 10.1080/10407413.2021.1965480 is OK
- 10.1098/rsif.2013.0475 is OK
- 10.7554/eLife.41703 is OK
- 10.1162/neco_a_00999 is OK
- 10.1016/j.neubiorev.2021.09.038 is OK
- 10.1016/j.schres.2021.07.032 is OK
- 10.1038/s41598-021-91308-x is OK

MISSING DOIs

- 10.3389/fpsyg.2022.855074 may be a valid DOI for title: Enactive-Dynamic Social Cognition and Active Inference

INVALID DOIs

- None
emdupre commented 2 years ago

Thank you, @conorheins ! I have one last reference request, since the included Hipólito reference was published in Frontiers over the weekend. Can you please update this reference to:

@article{hipolito2021enactive,
author = {Hip{\'o}lito, In{\^e}s and van Es, Thomas},
title = {Enactive-Dynamic Social Cognition and Active Inference},
journal = {Frontiers in Psychology},
volume = {13},
year = {2022},
url = {https://www.frontiersin.org/article/10.3389/fpsyg.2022.855074},
doi = {10.3389/fpsyg.2022.855074},
issn = {1664-1078}
}
conorheins commented 2 years ago

Hi @emdupre , I removed that reference as well as several others that weren't used here (some of them were left over from the arxiv version). Thanks

emdupre commented 2 years ago

@editorialbot check references

editorialbot commented 2 years ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1371/journal.pone.0006421 is OK
- 10.1162/neco_a_01427 is OK
- 10.1162/neco_a_01354 is OK
- 10.1080/17588928.2015.1020053 is OK
- 10.1038/s41598-018-38246-3 is OK
- 10.1016/j.neuroimage.2004.07.041 is OK
- 10.1007/s00422-012-0512-8 is OK
- 10.1016/j.tics.2011.11.018 is OK
- 10.1162/neco_a_01357 is OK
- 10.1109/IJCNN48605.2020.9207382 is OK
- 10.1162/NECO_a_00912 is OK
- 10.3390/e21030257 is OK
- 10.3389/frobt.2019.00020 is OK
- 10.1007/s00422-018-0785-7 is OK
- 10.1371/journal.pcbi.1007805 is OK
- 10.1109/ICASSP40776.2020.9054364 is OK
- 10.1016/B978-0-12-372560-8.X5000-1 is OK
- 10.1038/nrn2787 is OK
- 10.1016/j.jmp.2020.102447 is OK
- 10.1016/S0004-3702(98)00023-X is OK
- 10.1162/neco_a_01351 is OK
- 10.1093/cercor/bhu159 is OK
- 10.1016/j.drugalcdep.2020.108208 is OK
- 10.1503/jpn.200032 is OK
- 10.1016/j.jmp.2021.102632 is OK
- 10.1016/j.neuroimage.2008.02.054 is OK
- 10.1016/j.jmp.2020.102348 is OK
- 10.1007/978-3-030-64919-7_1 is OK
- 10.1016/j.ijar.2018.11.002 is OK
- 10.1098/rstb.1980.0090 is OK
- 10.1016/j.neubiorev.2016.06.022 is OK
- 10.1162/neco.1995.7.5.889 is OK
- 10.1037/13536-000 is OK
- 10.1007/978-3-030-93736-2_48 is OK
- 10.1016/j.neunet.2021.05.010 is OK
- 10.1109/LRA.2021.3090015 is OK
- 10.1016/B978-0-323-88506-5.50066-8 is OK
- 10.1007/s00422-019-00805-w is OK
- 10.3390/e23020198 is OK
- 10.1080/10407413.2021.1965480 is OK
- 10.1098/rsif.2013.0475 is OK
- 10.7554/eLife.41703 is OK
- 10.1162/neco_a_00999 is OK
- 10.1016/j.neubiorev.2021.09.038 is OK
- 10.1016/j.schres.2021.07.032 is OK
- 10.1038/s41598-021-91308-x is OK

MISSING DOIs

- None

INVALID DOIs

- None
emdupre commented 2 years ago

@openjournals/joss-eics, this is ready for final review !

kyleniemeyer commented 2 years ago

@editorialbot recommend-accept

editorialbot commented 2 years ago
Attempting dry run of processing paper acceptance...
editorialbot commented 2 years ago
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):

OK DOIs

- 10.1371/journal.pone.0006421 is OK
- 10.1162/neco_a_01427 is OK
- 10.1162/neco_a_01354 is OK
- 10.1080/17588928.2015.1020053 is OK
- 10.1038/s41598-018-38246-3 is OK
- 10.1016/j.neuroimage.2004.07.041 is OK
- 10.1007/s00422-012-0512-8 is OK
- 10.1016/j.tics.2011.11.018 is OK
- 10.1162/neco_a_01357 is OK
- 10.1109/IJCNN48605.2020.9207382 is OK
- 10.1162/NECO_a_00912 is OK
- 10.3390/e21030257 is OK
- 10.3389/frobt.2019.00020 is OK
- 10.1007/s00422-018-0785-7 is OK
- 10.1371/journal.pcbi.1007805 is OK
- 10.1109/ICASSP40776.2020.9054364 is OK
- 10.1016/B978-0-12-372560-8.X5000-1 is OK
- 10.1038/nrn2787 is OK
- 10.1016/j.jmp.2020.102447 is OK
- 10.1016/S0004-3702(98)00023-X is OK
- 10.1162/neco_a_01351 is OK
- 10.1093/cercor/bhu159 is OK
- 10.1016/j.drugalcdep.2020.108208 is OK
- 10.1503/jpn.200032 is OK
- 10.1016/j.jmp.2021.102632 is OK
- 10.1016/j.neuroimage.2008.02.054 is OK
- 10.1016/j.jmp.2020.102348 is OK
- 10.1007/978-3-030-64919-7_1 is OK
- 10.1016/j.ijar.2018.11.002 is OK
- 10.1098/rstb.1980.0090 is OK
- 10.1016/j.neubiorev.2016.06.022 is OK
- 10.1162/neco.1995.7.5.889 is OK
- 10.1037/13536-000 is OK
- 10.1007/978-3-030-93736-2_48 is OK
- 10.1016/j.neunet.2021.05.010 is OK
- 10.1109/LRA.2021.3090015 is OK
- 10.1016/B978-0-323-88506-5.50066-8 is OK
- 10.1007/s00422-019-00805-w is OK
- 10.3390/e23020198 is OK
- 10.1080/10407413.2021.1965480 is OK
- 10.1098/rsif.2013.0475 is OK
- 10.7554/eLife.41703 is OK
- 10.1162/neco_a_00999 is OK
- 10.1016/j.neubiorev.2021.09.038 is OK
- 10.1016/j.schres.2021.07.032 is OK
- 10.1038/s41598-021-91308-x is OK

MISSING DOIs

- None

INVALID DOIs

- None
editorialbot commented 2 years ago

:wave: @openjournals/joss-eics, this paper is ready to be accepted and published.

Check final proof :point_right: https://github.com/openjournals/joss-papers/pull/3194

If the paper PDF and the deposit XML files look good in https://github.com/openjournals/joss-papers/pull/3194, then you can now move forward with accepting the submission by compiling again with the command @editorialbot accept

kyleniemeyer commented 2 years ago

Hi @conorheins, I'm the AEIC on duty this week doing some final checks.

In your article, can you remove the explicit arXiv URL from the end of the statement of need section? The citation you included is sufficient and includes the URL.

Similarly, the article already links to the software repository, so I think you should remove the URL pointing to that in the same paragraph. If you really want to keep a link in the text, perhaps just make the word "repository" a hyperlink like [repository](https://github.com/infer-actively/pymdp).

In the Introduction first paragraph, can you wrap "e.g." with commas, like "... in, e.g., psychophysics ...". Similarly, please add a comma after "e.g." in later uses.

It looks like the Parr et al. 2020 article from Cerebral Cortex is missing a DOI, can you check if one is available?

conorheins commented 2 years ago

Hi @kyleniemeyer, thanks for the final checks. I have made the changes you requested, let me know if anything else needs fixing.

kyleniemeyer commented 2 years ago

@editorialbot generate pdf

editorialbot commented 2 years ago

:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left: