Unable to provide fast (eg dt=0.1) timestamps for amplitude injection

Open
  • Is something wrong?
#429 2 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
38/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
numpy, python
Domain
backend

Research direction

Start at the IClamp module's CSV timestamp validation and reproduce the error with timestamps spaced by 0.1 ms, comparing the result with 1 ms spacing. Done means valid sub-millisecond timestamp intervals are accepted without the consistency error while the existing input behavior remains intact.

Written by the indexing model from the issue text.

Description

Hi there,

I am new to using BMTK, and I am trying to run a model where I inject a noisy current step using the IClamp module. I want this current injection to be a time varying stimulus with 0.1 ms steps. However, when I try to provide this resolution of time steps I get an error "Exception: IClampMod: csv timestamps column (timestamps) must have a consistent intervals." I think I have narrowed it down to how Python is representing floating points, and that there is an accumulation of errors across the timestamps vector such that when you run np.unique(np.diff(timestamps)), there are several different "interval" values. If I make my timestamps vector in increments of 1 (so dt=1ms), then I don't have this issue.

Ideally, I would inject a time varying stimulus defined by smaller timestamps than 1ms, though. Is there any way to allow for this functionality with the current set up?

Thank you!

Dominant language
Python
Stars
339
Forks
103
Avg merge
2h 7m
Merged PRs (30d)
14

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from AllenInstitute/bmtk

All issues in AllenInstitute/bmtk

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.