-
Notifications
You must be signed in to change notification settings - Fork 0
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
WIP: cleaning and revisiting spDCM tutorial #543
Merged
Merged
Changes from 6 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
fefe474
remove some lines of cod that are not used anymore but were forgotten…
david-hofmann 8ba8c82
Introduce rescaling of data and set absent links to zero + WIP: exper…
david-hofmann 5a3a8d8
WIP: stimulus blox
david-hofmann 6990feb
added ARBlox to emulate SPM noise source
david-hofmann f18229b
WIP: system still unbalanced
david-hofmann a59005f
not solved yet, simulation does not work
david-hofmann 097215e
fixed OUBlox jcn variable: the input changes the mean and needs to be…
david-hofmann cb8238b
Too many callbacks take too long to assemble the ODEProblem. New appr…
david-hofmann 1e995ec
Solution for ARBlox found. However, problem of inverted parameters pe…
david-hofmann b0217c8
Update src/blox/stochastic.jl
david-hofmann 340f10f
just removing some whitespaces
david-hofmann 990a5ef
fixed issue: simulation used standard Neuroblox connection matrix def…
david-hofmann 3d1b53e
tuned parameters to get better results. Note that an AR process noise…
david-hofmann b16b7f9
added some more explanations related to the CSD
david-hofmann a68a5df
add `color` kwarg to plot recipes that didn't have it (#538)
harisorgn 6d416bd
fix the lateral inhibition issue
anandpathak31 488792c
fixing sequence of discrete events
anandpathak31 22c5be7
Callback and connection fixes for CS model (#542)
harisorgn 28a6628
Initial VdP moving
agchesebro 0bda3a1
Split noise vs no noise VdP and add constructor
agchesebro ce8f4da
Cleanup and add test
agchesebro 59616f7
remove old VdP test
MasonProtter f2ce6fe
Remove CairoMakie from deps
agchesebro ea4302b
ah yes well I'm dumb
agchesebro 2ebdab9
integrate Van der Pol with GraphDynamics and test it
MasonProtter d205fc2
Update Kuramoto to new system
agchesebro 2cc466d
New Kuramoto test
agchesebro 654b6cb
`return` not parsed in ternary operator
agchesebro 5901137
Remove old test
agchesebro 88eb9fb
fix bug in noisy Kuramoto connection
MasonProtter 5a44153
adapt GraphDynamics code to Kuramoto cleanup
MasonProtter 310674e
make neural-mass housekeeping PR non-breaking (#532)
MasonProtter 23e33f9
remove unneeded comments
MasonProtter a9b54fe
temporarily disable peak-detection tests for MetabolicHHNeuron networ…
bbantal 29d3f44
Solved issue with new MTK parameter structure, removed split=false & …
david-hofmann 7f7c9ac
some more edits to comments
david-hofmann 01b266d
Merge branch 'master' into cleanup
david-hofmann File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@harisorgn I introduced this equation because otherwise it would complain that the system is unbalanced. However, I am a bit concerned that this way we will produce impulse like perturbations of the system at each t_stim point after which x will again be set to 0? Not sure but once we can simulate it we will know, perhaps you know already the expected behavior.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the solution is to remove the initial guess
=0.0
from yourx(t)
state, but in general you could also definex
as a parameter, e.g. this is what we have done inImageStimulus
as it is only affected by callbacks, it doesn't have dynamics.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, that's not true here. It would work if
x
was just some placeholder state likejcn
that is simplified away. But here you want to keep itirreducible=true
so making it a parameter is my recommended solution. Otherwise you will end up with a DAE system.