search

paris-saclay-cds / ramp-workflow

Toolkit for building predictive workflows on top of pydata (pandas, scikit-learn, pytorch, keras, etc.).
https://paris-saclay-cds.github.io/ramp-docs/
BSD 3-Clause "New" or "Revised" License
68 stars 43 forks source link

MAINT: replace os.path with pathlib.Path #317

Open rth opened 1 year ago

rth commented 1 year ago

This package does a lot of path manipulations, and it would be nice to use pathlib.Path more. See https://treyhunner.com/2018/12/why-you-should-be-using-pathlib/ for motivation of using pathlib.

This is particularly useful in combination with typing #316 since path then would be of type Path instead of str. The code would also be much sorter (less of os.path.join everywhere) and more readable.

A potential maintenance task cc @SofianChay

kegl commented 1 year ago

I second it.