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
Projections #150
Projections #150
Conversation
brion/blueConfig.h
Outdated
|
||
/** | ||
* @return the full path to the morphology database. A suffix may be | ||
* prepended to the to the bare path from the BlueConfig. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to the to the?
Hold on. I didn't see the 3rd commit while reviewing...
brain/circuit.h
Outdated
* to cells in the projecting circuit. | ||
* \endif | ||
*/ | ||
BRAIN_API SynapsesStream getAfferentProjections( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't get a confusion with the existing getProjectedSynapses() in terms of similar names I hope.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Difficult to say, if you have a better suggestion it will be welcomed.
brain/detail/synapsesStream.h
Outdated
_afferent, _prefetch); | ||
}); | ||
} | ||
else |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
else not needed
brain/synapses.cpp
Outdated
@@ -123,14 +139,18 @@ struct Synapses::Impl : public Synapses::BaseImpl | |||
return; | |||
|
|||
const brion::Synapse& synapseAttributes = | |||
_circuit._impl->getSynapseAttributes(_afferent); | |||
const brion::Synapse* synapseExtra = _circuit._impl->getSynapseExtra(); | |||
_source.empty() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename _source to _externalSource or similar? It's not quite obvious deeply in the code what this means.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll do
c1ccdec
to
9a7e4f0
Compare
This function gives access to synape data for afferent projections external to the circuit.
No description provided.