Closed lepodl closed 3 years ago
Hey @lepodl,
Thanks for reporting here! As far as I can see, the implementation is consistent with the desired behavior. From the second-to-last paragraph of the materials and methods section in (Schmidt et al. 2018): "Therefore, we introduce two factors chi
and chi_I
to scale the weight of synapses onto excitatory neurons by chi
and onto inhibitory neurons by chi_I
[emphasis added]."
In the above code, cc_weights_I_factor
is precisely this factor chi_I
. Do you agree that the implementation is consistent with the manuscript or am I missing something here? Maybe it is important to mention here that a small fraction of cortico-cortical projections in the model target inhibitory neurons.
Cheers, Alex
PS: Larger blocks of code can be enclosed by ``` to make them better legible, see e.g. here.
Hi, I agree with @AlexVanMeegen , I think the implementation is in line with the manuscript. To make the code snippet more readable, I edited @lepodl 's comment to add syntax highlighting and proper code formatting.
Okay, it looks like I made a mistake, thank you!!
Most welcome! Thanks again for reporting, certainly better to double check than having a bug. I'll go ahead and close the issue.
Is "for 'i' in target_pop" an error in this code in Model.py? Inhibitory weights are enhanced by factor cc_weights_I_factor, but you choose the inhibitory target_pop as maniuplating object.