hnn-core icon indicating copy to clipboard operation
hnn-core copied to clipboard

bursty drive seeding discrepancy

Open rythorpe opened this issue 3 years ago • 6 comments

We should look into this more and determine if the discrepancy is indeed due to random seed variation. If so, we should consider warning users to be weary of using single trials.

Side note, I think I remember @ntolley mentioning that L2 basket cell spiking is very sensitive to the random state in Rob Law's model. We should resolve this or come up with a game plan for how we will address variation in single trials in the future.

rythorpe avatar Jul 22 '21 20:07 rythorpe

Commenting to confirm that is indeed the case, the original simulation parameters rely heavily on coincident spike times to just barely bring the L2 Basket cells to their firing threshold. This can be made simpler by changing the input weights, but it would no longer be a replication of the original model parameters of the study.

ntolley avatar Jul 23 '21 05:07 ntolley

one way to get around this issue is the following:

set the std deviation of the timing of spikes to 0, so you get the precise times you want. Then say that if you want more realistic activity, increase the std deviation and n_trials. The neuron simulations are all deterministic so the only stochasticity comes from the input spikes. That way we're more explicit rather than setting the seed which implicitly does the same

jasmainak avatar Jul 23 '21 15:07 jasmainak

We need some spike time variation (or at least dispersion) to get a beta event. Are suggesting that we hardcode a spike train that we feed to the population of drive cells?

rythorpe avatar Jul 23 '21 16:07 rythorpe

I see, and do we also need burst_std > 0 for beta?

jasmainak avatar Jul 23 '21 16:07 jasmainak

I'll play around a bit to see if something can be done while getting somewhat more consistent results across seeds :)

jasmainak avatar Jul 23 '21 16:07 jasmainak

I see, and do we also need burst_std > 0 for beta?

tstart_std can be 0 (which is how it's already implemented in the beta example I believe), but we need burst_std > 0.

rythorpe avatar Jul 23 '21 16:07 rythorpe