search

BlueBrain / morph-tool

https://morph-tool.readthedocs.org/
GNU Lesser General Public License v3.0
8 stars 7 forks source link

extend neurondb to dataframe for multiple path columns #32

Closed arnaudon closed 4 years ago

arnaudon commented 4 years ago

Allow to give a dict to neurondb -> dataframe loader to have multiple path columns (for file format, morphology versions, etc...)

arnaudon commented 4 years ago

arg, sorry, I was using gh, but it messed up with what was on my branch, let me do that PR again...

wizmer commented 4 years ago

Changing the signature to this function would break this API as it is already used in multiple places. I won't be the way to go. Why not a different function though ?

wizmer commented 4 years ago

But what you want is simply something like:

pd.concat(neurondb_dataframe(neurondb, path) for path in paths)

No ?

arnaudon commented 4 years ago

ah ok, I can do it on my side then, I thought this could be of the use here, but if people already use it like that no problemo!

arnaudon commented 4 years ago

yes, I can do something like that, with a merge, not concat, I want same number of rows, but several columns for paths