arbor-sim / arbor

The Arbor multi-compartment neural network simulation library.
https://arbor-sim.org
BSD 3-Clause "New" or "Revised" License
105 stars 59 forks source link

Update `segment` to `msegment` as name present in arbor module #2237

Closed Helveg closed 7 months ago

Helveg commented 7 months ago

It seems the segment class has been renamed to msegment, I updated the morphology docs to reflect this (probably breaking a couple labels elsewhere?)

thorstenhater commented 7 months ago

Interesting question, I stumbled on this during #2233, we might want to rename:

which are either currently divergent or inconsistent. I personally dislike the namespacing on a budget introduce by the m prefix.

thorstenhater commented 7 months ago

Based on our discussion in #2239, let's take this and pull the rest up this convention.