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

Can membrane potential be recorded when a cell is of type point_process? #55

Closed
pgleeson opened this issue Oct 16, 2018 · 4 comments
Closed
Assignees
Labels
question simulation config Simulation config json

Comments

@pgleeson
Copy link
Contributor

If I have a population where the cells are point_process, it should be possible to record the membrane potential (in order to check the sub threshold v in addition to saving spikes). I've tried this:

  "reports": {
    "membrane_potential": {
      "cells": "point_nodes",
      "variable_name": "v",
      "module": "membrane_report",
      "sections": "soma"
    }
  }

with an entry in node_sets.json:

{
  "point_nodes": {
    "model_type": "point_process"
  }
}

and it does save a output/membrane_potential.h5, but the data array is empty

Code for this model here: https://github.com/pgleeson/sonata/tree/intfire/examples/small_intfire

@hernando
Copy link
Collaborator

Shouldn't this ticket go to another repository?
By the way, as per #33 point_process is going to be renamed to point_neuron. I don't know if this change will be done in coordination with other tools.

@pgleeson
Copy link
Contributor Author

I agree it may be related to BMTK, but it probably should be raised here, as the reports block is part of the spec, and it's relevant for the examples included for Sonata.

I did try "variable_name": "m", as suggested by the Neuron docs on IntFire1, but that didn't work. https://www.neuron.yale.edu/neuron/static/docs/help/neuron/neuron/mech.html#IntFire1.

This is probably related to the bigger issue of allowing (and using in standard examples) simulator specific cell types, e.g. for a NEST simulation the line will be: "variable_name": "V_m" as here.

Moving to PyNN or NeuroML based cell definitions in the longer term will help address this issue.

@kaeldai
Copy link
Collaborator

kaeldai commented Oct 17, 2018

I can confirm that in the bmtk we don't record cell variables of NEURON's intfire cells, and just skip those cells without warning. We can add recording of intfire cells, it's just something no one yet had asked for.

I like the idea of adding a dictionary of terms for specifying attributes like voltage, calcium, etc. And I'd prefer to use that which has already been done in NeuroML2 or PyNN. I'd suggest we create a new enhancement request for SONATA, and move this bug to the bmtk repo. However in the short term I'd recommend we can add language suggesting that a given implementation give a warning and/or exit out if the user tries to report on a cell variable that doesn't exists.

@pgleeson
Copy link
Contributor Author

No longer relevant as nrn:IntFire1is fundamentally tied to an inbuilt Neuron artificial cell...

It should be possible to record v from an I&F cell when #91 is implemented

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question simulation config Simulation config json
Projects
None yet
Development

No branches or pull requests

3 participants