Skip to content
This repository was archived by the owner on Mar 20, 2023. It is now read-only.

Pass PI value from NEURON to CoreNEURON - #494

Merged
bdelmarm merged 1 commit into
sandbox/bdelmarm/lfp_rebasefrom
magkanar/lfp_rebase_pi_fix
Feb 12, 2021
Merged

bdelmarm merged 1 commit into
sandbox/bdelmarm/lfp_rebasefrom
magkanar/lfp_rebase_pi_fix

Conversation

@iomaganaris

Copy link
Copy Markdown
Contributor

Description

In the lfp PR during the calculation of lfp there is a variable PI used. The value of this variable should be same as NEURON and the variable used by the mod files.

  • Created new pi global parameter which is read by NEURON globals.dat or directly NEURON in online mode
  • Use this pi in lfp calculation and test
  • Removed a deprecated boost test header file

- Created new pi global parameter which is read by NEURON globals.dat or
  directly NEURON in online mode
- Use this pi in lfp calculation and test
- Removed a deprecated boost test header file
Comment thread coreneuron/io/global_vars.cpp
Comment thread tests/unit/lfp/lfp.cpp
@pramodk

pramodk commented Feb 11, 2021

Copy link
Copy Markdown
Collaborator

@bdelmarm : feel free to merge this PR (it will be merged in your PR).

@bdelmarm bdelmarm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great @iomaganaris . Thank you!

@bdelmarm
bdelmarm merged commit 8bcc10a into sandbox/bdelmarm/lfp_rebase Feb 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants