AllenInstitute / AllenSDK

code for reading and processing Allen Institute for Brain Science data
https://allensdk.readthedocs.io/en/latest/
Other
344 stars 150 forks source link

transition to pathlib #675

Open NileGraddis opened 5 years ago

NileGraddis commented 5 years ago

pathlib offers a nifty object-oriented API and lots of convenience functionality for interacting with fs paths. See the pep for more info.

We can probably replace some or all of our path-munging code with pathlib. It is, however python 3 only.

NileGraddis commented 5 years ago

I've switched brain_observatory/ecephys to pathlib on branch 673 as an experiment. Thoughts thus far:

njmei commented 5 years ago

Hmm... yeah it looks like for your use case it might not make as much sense. For me, the strengths of pathlib are:

But yeah, was only a suggestion, I think we can adopt and use on a case-by-case basis. It should be another tool in our toolbox.