devopsarr / radarr-py

python SDK for Radarr
Mozilla Public License 2.0
4 stars 2 forks source link

Cannot import module in venv #51

Closed ryanwalder closed 7 months ago

ryanwalder commented 9 months ago

I've installed the module in a venv but am getting errors when trying to import it:

  1. python -m venv /tmp/foo
  2. cd /tmp/foo
  3. source bin/activate
  4. pip install git+https://github.com/devopsarr/radarr-py.git
  5. python
  6. (in python) import radarr

Error:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/tmp/foo/lib/python3.11/site-packages/radarr/__init__.py", line 22, in <module>
    from radarr.api.alternative_title_api import AlternativeTitleApi
  File "/tmp/foo/lib/python3.11/site-packages/radarr/api/__init__.py", line 48, in <module>
    from radarr.api.log_file_api import LogFileApi
  File "/tmp/foo/lib/python3.11/site-packages/radarr/api/log_file_api.py", line 22, in <module>
    from pydantic import field_validator
ImportError: cannot import name 'field_validator' from 'pydantic' (/tmp/foo/lib/python3.11/site-packages/pydantic/__init__.cpython-311-x86_64-linux-gnu.so)

venv packages:

/tmp/foo $ pip list
Package            Version
------------------ ----------
aenum              3.1.15
certifi            2023.11.17
charset-normalizer 3.3.2
idna               3.6
pip                23.2.1
pydantic           1.10.13
python-dateutil    2.8.2
radarr-py          0.4.0
requests           2.31.0
setuptools         68.2.2
six                1.16.0
typing_extensions  4.9.0
urllib3            2.1.0
wheel              0.41.2
davidgibbons commented 9 months ago

I get the same error, it looks like maybe changes were made to support pydantic 2 that aren't backwards compatible. If I pip install pydantic==2 I can get past the import error, although I get field required errors instead. It's at least a code exception instead of a import error. It seems like the older 0.4.0 released version on pypi doesn't have that issue but it also seems out of date compared to the api.

Fuochi commented 7 months ago

Hi @davidgibbons @ryanwalder, sorry for the late response. I updated the sdk to pydantic v2, it should work now.

ryanwalder commented 7 months ago

@Fuochi No worries for the late response, thanks for taking the time to update things. Just given it a quick test (just loading the module) and it works as expected. Thanks again!