Skip to content

Comments

Implementation of Wacongne et al. 2012 paper with using Brian2.#1767

Open
cagataydisli wants to merge 5 commits intobrian-team:masterfrom
cagataydisli:master
Open

Implementation of Wacongne et al. 2012 paper with using Brian2.#1767
cagataydisli wants to merge 5 commits intobrian-team:masterfrom
cagataydisli:master

Conversation

@cagataydisli
Copy link

Me (@cagataydisli) and my friend (@AtakanDogan21) have implemented Wacongne's 2012 paper about Predictive Coding and Mismatch Negativity. We want to contrubute to the Brian2 documantation.

@mstimberg
Copy link
Member

Hi @cagataydisli and @AtakanDogan21, thank you for your PR (and apologies for not replying timely on the previous one)! This is a big example, and I did not yet have time to look into it in detail, but it also makes me wonder whether the inclusion in the documentation is actually the best format for such an example. Historically, "from_paper" examples were single-file scripts, and multi-file examples were mostly about generating one figure per file. Your structure makes certainly sense (from the very cursory look I had, at least), but it will not be obvious to use from the documentation (e.g. you cannot download all files at once). I wonder whether it would not make more sense to have a link to a separate repository or something similar in such cases. I'll raise this issue on the discourse forum, let's see what others think. So in sum, thanks again, complex examples like this are much appreciated, but I am putting this PR "on hold" for now until we decide what the best approach for including it into Brian's documentation is.

PS: I think for copyright reasons, you should remove the paper PDF from the PR – we wouldn't include it in the documentation, in any case.
PPS: I don't know whether you've considered this, but it might be of interest to publish a short article with your code in the Rescience-C journal: https://rescience.github.io/

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.

2 participants