[MRG] fix layers 5 and 2 swapped depth dimension

[cjayb]

Closes #351

[codecov-commenter]

Codecov Report

Merging #352 (ad09cf4) into master (c65bc1e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #352   +/-   ##
=======================================
  Coverage   90.94%   90.94%           
=======================================
  Files          13       13           
  Lines        2540     2540           
=======================================
  Hits         2310     2310           
  Misses        230      230           

Impacted Files	Coverage Δ
hnn_core/network.py	93.14% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c65bc1e...ad09cf4. Read the comment docs.

[cjayb]

Hmm, digging a little deeper, what's this doing @rythorpe:

hnn-core/hnn_core/cell.py

Line 319 in c65bc1e

dy = self.pos[2] - self.p_secs['soma']['sec_pts'][0][1]

So net.pos_dict contains coordinates such that apical=Z, and Cell flips Z and Y so that apical sections are along Y?

Man it's hard to keep track of all these flips splattered all over the place! I'm honestly uncertain which way our dendrite section s are pointing!

[jasmainak]

yeah, this is what I saying here @rythorpe

[jasmainak]

Thanks @cjayb ! Feel free to add a whats_new entry for this in your LFP pull request.

[rythorpe]

Just getting caught up - good catch here @cjayb! Looks like @jasmainak is dismantling the tyranny of incongruent morphology orientation in his most recent PR!