plotting still uses channel_index

NeuralEnsemble / PyNN

A Python package for simulator-independent specification of neuronal network models.

Other

276 stars 126 forks source link

plotting still uses channel_index #708

Closed Christian-B closed 2 years ago

Christian-B commented 3 years ago

According to https://neo.readthedocs.io/en/stable/releases/0.9.0.html We should no longer be using channel_index.

Yes https://github.com/NeuralEnsemble/PyNN/blob/master/pyNN/utility/plotting.py Still references channel_index

jiegec commented 3 years ago

Neo 0.10.0 is released and ChannelIndex is removed. A fix is needed for neo 0.10.0 support. @apdavison

apdavison commented 2 years ago

most of the changes related to the removal of ChannelIndex from Neo were in #732. plotting.py was fixed in bbf0e206.

apdavison commented 2 years ago

Note that you will still find reference to "channel_index", but this is now just an array annotation added by PyNN here to facilitate plotting/data analysis.

Christian-B commented 2 years ago

Thank you. My recent comment have been deleted as they where not correct. Apologies.