rstudio / reticulate

R Interface to Python
https://rstudio.github.io/reticulate
Apache License 2.0
1.65k stars 326 forks source link

config.py bug #1493

Closed Aldohlys closed 9 months ago

Aldohlys commented 9 months ago

Looking into the config.py source code, I notice the following test: if sys.version < '3.4' But then if sys.version='3.12' for instance this test returns True (as it is string comparison and not a decimal comparison), which I guess is NOT the desired behaviour. This test should be replaced by:

if sys.version_info < (3, 4)

This will solve some problems - My problem which is I cannot import ib_insync module using reticulate is still opened, I posted this problem on RStudio Community forum.

Aldohlys commented 9 months ago

Looking at the Github repo, it seems that this was fixed 3 days ago. I just do not have the latest version.