dbbs-lab / models

Collection of neuron models for the NEURON simulator
GNU General Public License v3.0
3 stars 7 forks source link

Synaptic situation #5

Closed SteMasoli closed 3 years ago

SteMasoli commented 3 years ago

The AMPA_PF and AMPA_AA are the same, with the same parameters and same MOD file used in PF PC and PF -SC

AMPA_PF is alone. AMPA_AA is used with NMDA stellate AMPA_MF is used with NMDA stellate

Sections for PF

if sec_index >= 4 and sec_index <= 15 or sec_index >= 18 and sec_index <= 32 or sec_index >= 42 and sec_index <= 83 or sec_index >= 85 and sec_index <= 104:

Sections for MF and AA.

if sec_index >= 108 and sec_index <= 112 or sec_index >= 114 and sec_index <= 121 or sec_index >= 128 and sec_index <= 129 or sec_index >= 131 and sec_index <= 132 or sec_index >= 135 and sec_index <= 140 or sec_index >= 144 and sec_index <= 145 or sec_index >= 147 and sec_index <= 150:

https://github.com/dbbs-lab/models/issues/2

To summarize.

Golgi AMPA_PF/AA - PC, SC, BC are the same mod with different "external" values". Golgi AMPA_MF - The MOD is the one for the granule cell and it is valid only for MF-GrC and MF-GoC

Golgi NMDA_AA and NMDA_MF - Stellate NMDA

Granule cell NMDA - Only used for granule cells"

Helveg commented 3 years ago

Ok! Could you also correct the synapses list on the different section_types (basal dendrites, apical dendrites, ...) so that we know on which type of sections these synapses can occur? I see that you've included lists of IDs of the sections, those probably correspond to some label defined here: (?)

https://github.com/dbbs-lab/models/blob/aaa1d3a468d90b35163b8b479a778b91e057e388/dbbs_models/golgi_cell_models.py#L110-L123

So the lists of IDs given will correspond to some label, and then in the section_type with that label you can allow the synapses you want by adding them to the synapses list. Is this clear?

codecov[bot] commented 3 years ago

Codecov Report

Merging #5 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   78.04%   78.04%           
=======================================
  Files           3        3           
  Lines          41       41           
=======================================
  Hits           32       32           
  Misses          9        9           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3b6fe4d...bf8ca1f. Read the comment docs.