search

TRI-ML / dgp

ML Dataset Governance Policy for Autonomous Vehicle Datasets
https://tri-ml.github.io/dgp/
MIT License
93 stars 63 forks source link

fix: allow end-uers to use more modern version of matplotlib dep #90

Closed tk-woven closed 2 years ago

tk-woven commented 2 years ago

matplotlib 3.0.1, currently required in requirements.txt, is not compatible with Python 3.8 which is the default python version for Ubuntu 20.0. Attempting installation will use the platform module and expect it has linux_distribution, but this has been moved to the distro module in modern Python.

I encountered this issue when installing DGP in a venv. Our framework in Woven uses Python 3.8.

Assuming Matplotlib respects semantic versioning, I expect these relaxed version requirements won't be a problem, but we ought to only allow different minor and patch versions.

This change is Reviewable

tk-woven commented 2 years ago

@quincy-kh-chen , @kuanleetri , Sorry for tagging you. I don't seem to be able to assign reviewers. Could you send this to the right people or recommend them to me, please?

tk-woven commented 2 years ago

Thank you, Kuan!