Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tutorial alpha user feedback: Resting state simulation using neural mass models #28

Open
sbrincat opened this issue Oct 25, 2024 · 0 comments
Assignees

Comments

@sbrincat
Copy link
Collaborator

Here is my specific feedback for the resting state tutorial...

Cell 5

  • Would be helpful to insert a comment briefly explaining what voltage_timeseries() is doing

Cells 5/6

  • It appears results are somewhat stochastic. I assume this is expected from the solution to the SDE problem, but users might be confused if their plots look different from the website. Can we set a random seed in here to make this repeatable? Alternatively, we should explicitly tell users to expect a different result each run.

Cell 7

  • Feels a bit out of place to end with the built-in connectivity matrix, rather than the simulation results. I’d suggest either moving this heatmap plot to cell 1/2 where the connections are generated, or maybe just plotting the connection and correlation p-value in side-by-side subplots in cell 6.
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

No branches or pull requests

2 participants