NeuromatchAcademy / course-content

NMA Computational Neuroscience course
https://compneuro.neuromatch.io
Creative Commons Attribution 4.0 International
2.65k stars 983 forks source link

Updates to Allen Visual Behavior notebooks #1112

Closed matchings closed 2 months ago

matchings commented 2 months ago

Updates to account for most recent version of AllenSDK which includes multiple stimulus blocks for each recording session. More info can be found under "data file changelog" here: https://allensdk.readthedocs.io/en/latest/visual_behavior_optical_physiology.html

Also a few additional references was added to the documentation in the markdown file for the Neurons page

marius10p commented 2 months ago

@iamzoltan do you know what this failed check means? We need this update as soon as possible so the Allen Institute projects can run.

matchings commented 2 months ago

@iamzoltan Let me know if there is anything i can do to help troubleshoot the checks failing, or anything else to test and validate the changes to the notebooks that were part of this PR. I know its cutting it close to the start of the course, so I will be available most of today (July 7) to help.

iamzoltan commented 2 months ago

@marius10p and @matchings yes this is cutting it very close to the course. Rushing like this is how things break. The process to make changes is as follows:

We will get this going as soon as possible.

p.s Do not make direct commits to main

yavorska-iryna commented 2 months ago

I can redo the steps.

iamzoltan commented 2 months ago

@yavorska-iryna I called the staging branch allen-hot-fix