ReScience / submissions

ReScience C submissions
28 stars 7 forks source link

[Re] Hippocampal Phase-Amplitude Coupling unearthed again #30

Closed hhentschke closed 4 years ago

hhentschke commented 4 years ago

This is a submission to the Ten Years Reproducibility Challenge.

Original article: Hentschke, H., Perkins, M.G., Pearce, R.A., & Banks, M.I. (2007) Muscarinic blockade weakens interaction of gamma with theta rhythms in mouse hippocampus. Eur.J.Neurosci., 26, 1642–1656.

PDF URL: https://github.com/hhentschke/ReScience_10yrReproChallenge/blob/master/paper/article.pdf Metadata URL: https://github.com/hhentschke/ReScience_10yrReproChallenge/blob/master/paper/metadata.yaml Code URL: https://github.com/hhentschke/ReScience_10yrReproChallenge

Scientific domain: Neuroscience Programming language: Matlab Suggested editor: Thomas Arildsen

rougier commented 4 years ago

@hhentschke Thanks for your submission and sorry for the delay.

@ThomasA @oliviaguest @benoit-girard @eroesch @gdetor Can yu handle this submission for the Ten Years Reproducibility Challenge (only 1 reviewer needed) ?

oliviaguest commented 4 years ago

Happy to take this on!

oliviaguest commented 4 years ago

👋 @TomDonoghue would you be interested in reviewing this?

TomDonoghue commented 4 years ago

Hey @oliviaguest - thanks for thinking of me!

The 10 years reproducibility challenge looks really interesting, and I would be happy to review this submission! Are there are specific guidelines for reviewing this kind of article / submission that I should be aware of?

oliviaguest commented 4 years ago

Fantastic, @TomDonoghue — thank you! I actually am not sure if there are specific ones for this type [please fill me in @khinsen @rougier @benoit-girard] but generally, see this: http://rescience.github.io/edit/

oliviaguest commented 4 years ago

Hey @TomDonoghue — no pressure at all — but would you be able to give a rough ETA for your review? ☺️

TomDonoghue commented 4 years ago

Definitely this week! Current plan should have it done by Wednesday!

TomDonoghue commented 4 years ago

Okay, so I have read through the original paper, the submitted ReScience paper, and looked through the repository. This paper reports on an attempt to re-run the analyses from a paper from 2007 examining theta & gamma oscillations, and cross frequency interactions, in mouse hippocampus, while manipulating muscarinic acetylcholinergic activity with atropine. I think this is a very interesting and overall good submission for the 10 years reproducibility challenge. I commend the author on picking up this very interesting challenge, and doing the work to step through old code, organize it, and try to get it all working! I have some comments, mainly addressed at hopefully trying to clarify the narrative of the paper, and a couple comments / questions about the organization of the repository.

Review Related Comments

I have a couple open questions about the review process, that perhaps @oliviaguest can help with a bit. I read the general ReScience review comments, and the notes on this challenge in particular, but was left with a couple open questions.

Paper Comments

My main comment of the paper, is that I think the paper could be a bit clearer about what it tried to reproduce, and in what ways this was / was not successful.

Small / Stylistic Notes:

Code Notes

I do find the repository a little hard to understand what everything is. Could the layout of the files be described a bit more?

For example:

oliviaguest commented 4 years ago

@koustuvsinha @rougier can you shed some light on the questions above with respect to the review? Your input would be greatly appreciated.

hhentschke commented 4 years ago

@TomDonoghue Thank you very much for the very thorough review! I'll answer your points in detail once your review-related comments have been commented upon. Just a quick explanation of my take on the challenge. My understanding, garnered from the description of the challenge (https://rescience.github.io/ten-years/), was that a depiction of the journey was more important than the destination. Given that unsuccessful attempts at reproduction were admissable, provided the story is interesting, made me believe that a proof of concept of repeatability plus a selected choice of actual figure reproductions would be adequate. Also, compared to a few early submissions I peeked at, which were quite short (2-4 pages), my manuscript is lengthy; showing reproductions of all nine figures of the orignal paper in the manuscript would certainly be off-putting for readers. That being said, I agree that this goal should be communicated early on. Will do so.

TomDonoghue commented 4 years ago

Hey @hhentschke - I agree, and sorry, I didn't mean to imply that I think the paper should have to include all of the original figures! Related to that point my main suggestion would be describe a bit more explicitly the plan and sub-selection chosen for replication. I also understand the call as more of a description of the replication attempt, and did scan and see at least one other submission that explicitly talked about replicating a subset of the paper. In terms of if there is a threshold for how much should be re-run, I'll leave that to the editors to say anything more about.

hhentschke commented 4 years ago

Hi @TomDonoghue, I decided to move forward and work on your comments, irrespective of the issue of completeness. You were quite right with the points you mentioned, and I changed the manuscript in the following ways:

As to the code and repository, I have reorganized, corrected and extended the documentation of the repository such that hopefully the structure of the code is much clearer now. In detail:

The repository does in fact contain the code in its original, unchanged version, including a commit message explaining its state. See the initial commit from April 3rd this year. Including the latest commit, there is a total of 15 commits (https://github.com/hhentschke/ReScience_10yrReproChallenge/commits/master), documenting the progression of changes.

Once again, thanks for your efforts! I'm curious to hear what the editors say.

oliviaguest commented 4 years ago

@TomDonoghue are you currently happy with the changes, etc.? ☺️

TomDonoghue commented 4 years ago

Hey @hhentschke (and cc: @oliviaguest):

First of all, I would like to apologize for the large delay in replying here. It seems I missed the original notification, and happened to notice this when scrolling through open issues, so my bad and my apologies for that.

I have revisited the project, and checked the updated repository and the updated paper. I am happy with the updates to the paper, and I find the organization of the updated version to be much easier to follow. I think the updates to the repository also help to organize and describe the files. When I re-read the paper now, I don't have any other notable comments or concerns about this paper.

In terms of my original questions to the editors - I think the updated paper is quite clear on replicating the usage of the a processing toolbox and a key set of results from the original paper, and I do not have any worries about the extent of the replication. As far as I'm aware, reviewing and/or re-running the code is not strictly required. Overall, I don't feel there is any outstanding issues that necessarily require anything more here.

All in all, I consider this paper is an interesting and useful submission for the 10 Years Challenge, as I understand the call, and in my capacity as reviewer, I would be happy to say this submission looks good to me to be accepted for that.

Thank you for the work on the updates @hhentschke - I enjoyed reading and reviewing this paper, congrats on the interesting submission!

@oliviaguest - please let me know if there is anything else you need from me for this submission!

hhentschke commented 4 years ago

@TomDonoghue, no problem about the delay - after all it's vacation time, and this is not a project with a deadline. Thank you very much for the time and effort you invested in the review, and your kind words!

rougier commented 4 years ago

@oliviaguest Gentle reminder

oliviaguest commented 4 years ago

@hhentschke do you have an update on the status of your submission, given the reviewer feedback? 😊

hhentschke commented 4 years ago

@oliviaguest I submitted a thorough revision on June 15th following the first review, which @TomDonoghue approved of about a month ago. I'm not familiar with the submission procedure - is there anything up to me right now?

oliviaguest commented 4 years ago

@hhentschke Yes, can you please fill in the missing details in the https://github.com/hhentschke/ReScience_10yrReproChallenge/blob/master/paper/metadata.yaml file, please? Set acceptance date as today. 🎉

This is relevant info: https://github.com/ReScience/ReScience/issues/48#issuecomment-689405707. And my ORCiD is: https://orcid.org/0000-0002-1891-0972 — let me know if anything is not clear. ☺️

hhentschke commented 4 years ago

Yay! Thanks a lot for your efforts! I left the last but one block ("article") in the metadata.yaml open because I'm not sure whether that final step of assigning article number, DOI and pdf url is up to you (including re-making the file) or me. Will do so if you tell me to.

oliviaguest commented 4 years ago

~@TomDonoghue what's your ORCID?~

No worries, found it.

oliviaguest commented 4 years ago

@rougier is this all done now? How do we update the website?

@hhentschke please merge https://github.com/hhentschke/ReScience_10yrReproChallenge/pull/1 😊 when you can!

rougier commented 4 years ago

I just updated the website and the entry is online now.

oliviaguest commented 4 years ago

@hhentschke yay! 🥳

Congratulations! http://rescience.github.io/bibliography/Hentschke_2020.html