Add problem set and solutions plus visual learning metadata - #6
Add problem set and solutions plus visual learning metadata#6matchings wants to merge 13 commits into
Conversation
There was a problem hiding this comment.
I'm not totally sure what's happening here, maybe updating the order in which these cells were run? I don't think you want to push this in the PR - can you discard the changes?
There was a problem hiding this comment.
Same comment as above - can you revert the changes made to all the metadata notebooks except for visual_learning? I think this will create a merge conflict later on.
There was a problem hiding this comment.
I think this code could be simplified. We should discuss further metadata fields to add to make these assets more searchable. Probably not enough time to do this, so we can push this through.
There was a problem hiding this comment.
I think we could cut part 1 as this is redundant with the tutorials and the group exercise we'll ask them to do. The problem set is quite long and I don't think we'll need to structure them through how to explore a dataset for a 3rd time. We could keep that bit in the solutions, as it could be a useful reference for the students later.
This PR adds the following things:
All solutions notebooks run in full if the metadata .csvs for all datasets are present.
There may be some additional changes to the exact conditions plotted in the BCI notebook and the exact example used for DR along with any other requested changes based on feedback. But this should be ready to release as is.
Capsule is here: https://codeocean.allenneuraldynamics.org/capsule/6206290/tree