frc-characterization icon indicating copy to clipboard operation
frc-characterization copied to clipboard

Docs are incorrect about python version

Open clrozeboom opened this issue 5 years ago • 8 comments

I tried using this with Python 3.6 and ran into an issue with importlib.resources not being found. I then updated to Python 3.8 and had problems installing robotpy-characterization. 3.7 seems to be the Goldilocks version, though the docs say 3.6 and greater.

clrozeboom avatar Oct 20 '19 18:10 clrozeboom

Yes, 3.8 is broken - statsmodels is unable to install, I discovered this independently last night. I'll update the dependency list and docs accordingly.

Oblarg avatar Oct 20 '19 18:10 Oblarg

Numpy hasn't made a release for 3.8 yet, so that's why it won't install.

calcmogul avatar Oct 20 '19 18:10 calcmogul

Numpy has released for 3.8 put matplotlib has not. Seems like the readmes should be updated to suggest exactly 3.7 until matplotlib 3.2 is released. The PyPI package specifies 3.7 so the current readme note about 3.6 definitely needs to be updated.

Kevin-OConnor avatar Nov 11 '19 17:11 Kevin-OConnor

Looks like statsmodels also doesn't have a release that works on 3.8 yet. This issue suggests that version 11 will likely support 3.8: https://github.com/statsmodels/statsmodels/issues/6220 The described workaround there did not work for me to get statsmodels running on 3.8

Kevin-OConnor avatar Nov 11 '19 17:11 Kevin-OConnor

The frc-docs articles about this state clearly that only Python 3.7 should be used. So this has been resolved and can be closed. If someone wants to check if the dependency now has a 3.8 version, go for it!

Starlight220 avatar Sep 16 '20 19:09 Starlight220

Both numpy and statsmodels seem to support Python 3.9 now. Are there any plans to support Python 3.9? I could help test anything that needs testing but I don't have access to a robot right now.

BobSaidHi avatar Apr 15 '21 21:04 BobSaidHi

This tool will be replaced with https://github.com/wpilibsuite/sysid for the 2022 season so there probably isn't a need to update it for Python 3.9 support.

Piphi5 avatar Apr 15 '21 21:04 Piphi5

The upgrade itself isn't hard, but frc-characterization has no unit tests or integration tests, so making sure we didn't break something more on Windows, macOS, or Linux is difficult.

calcmogul avatar Apr 15 '21 22:04 calcmogul