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

MODEL: Adjust cell spacing #968

Open
asoplata opened this issue Jan 14, 2025 · 2 comments · May be fixed by #985
Open

MODEL: Adjust cell spacing #968

asoplata opened this issue Jan 14, 2025 · 2 comments · May be fixed by #985
Labels
default-model-update Issues applicable to the subsequent iteration of the default HNN network model science

Comments

@asoplata
Copy link
Collaborator

cc: @ntolley @carolinafernandezp @katduecker

I don't know the details, but as Nick mentioned in the last dev meeting, there is a scientific issue having to do with placement of the cells in space. IIRC, the issue with the placement of the inhibitory and excitatory cells should not, in principle, affect the current-dipole output of our simulations, but it will likely affect any LFP/CSD output. In order for us to also work towards scientifically meaningful LFP/CSD, it sounded like this is an issue that needs to be addressed. I've made this issue for us to track our work.

@asoplata asoplata added default-model-update Issues applicable to the subsequent iteration of the default HNN network model science labels Jan 14, 2025
@asoplata
Copy link
Collaborator Author

Irresponsible suggestion: it may make sense to also deal with #342 and #568 in the course of solving this issue.

@carolinafernandezp
Copy link
Contributor

carolinafernandezp commented Jan 15, 2025

I can work on this. The problem is that the same-layer basket cells and pyramidal neurons should be on the same plane. Currently, the basket cells are arbitrarily placed slightly above (L5) and slightly below (L2) their respective pyramidal cell layers.

This is happening in _calc_basket_coord where zdiff (the depth of L2 relative to L5 pyramidal cell somas) gets offset by a weight of 0.8 and 0.2 for the L2 and L5 basket cells, respectively (which ends up being 261.48 μm since zdiff is hard-coded to 1307.4 μm).

I'm rerunning the LFP/CSD simulations to see how they are affected by removing the offset.

@carolinafernandezp carolinafernandezp linked a pull request Jan 30, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
default-model-update Issues applicable to the subsequent iteration of the default HNN network model science
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants