-
-
Notifications
You must be signed in to change notification settings - Fork 305
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LLM 0.13 broken on Windows #407
Comments
|
I really need to figure out a good Windows dev environment for myself. For the moment can I at least get CI running on Windows? |
Windows CI failed with that same error: https://github.com/simonw/llm/actions/runs/7674731683/job/20919846177 |
Doing the rest of the work in PR: |
* Run CI on Windows and macOS as well as Ubuntu, refs #407 * Use pyreadline3 on win32 * Back to fail-fast since we have a bigger matrix now * Mark some tests as xfail on windows
Released as 0.13.1: https://llm.datasette.io/en/stable/changelog.html#v0-13-1 |
Thanks for the fast turn-around. Working great now! |
I was using conda venv, and the fix was to install pydantic using conda.exe
From https://docs.pydantic.dev/latest/install/ because my llm instance was haivng an issue was pydantic instead of readline. (base) C:\Users\dance2die>llm
Traceback (most recent call last):
File "<frozen runpy>", line 198, in _run_module_as_main
File "<frozen runpy>", line 88, in _run_code
File "C:\Users\dance2die\anaconda3\Scripts\llm.exe\__main__.py", line 4, in <module>
File "C:\Users\dance2die\anaconda3\Lib\site-packages\llm\__init__.py", line 18, in <module>
from .plugins import pm
File "C:\Users\dance2die\anaconda3\Lib\site-packages\llm\plugins.py", line 17, in <module>
pm.load_setuptools_entrypoints("llm")
File "C:\Users\dance2die\anaconda3\Lib\site-packages\pluggy\_manager.py", line 287, in load_setuptools_entrypoints
plugin = ep.load()
^^^^^^^^^
File "C:\Users\dance2die\anaconda3\Lib\importlib\metadata\__init__.py", line 202, in load
module = import_module(match.group('module'))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\dance2die\anaconda3\Lib\importlib\__init__.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\dance2die\anaconda3\Lib\site-packages\llm_claude\__init__.py", line 6, in <module>
from pydantic import Field, field_validator
ImportError: cannot import name 'field_validator' from 'pydantic' (C:\Users\dance2die\anaconda3\Lib\site-packages\pydantic\__init__.cp311-win_amd64.pyd) |
Ended up with a similar stacktrace to the last comment on my M1 iMac. It's unrelated to the original reported issue. Google brought me here.
From here and a quick bit of Googling around, it looks like these fields are missing on (at least some versions of) pydantic 1.x Lo and behold what was I running:
An upgrade of brew's pydantic module got me up and running.
@simonw hope it's ok to use comments in this fashion here and you don't feel like it's polluting your issues! |
Reported here: https://discord.com/channels/823971286308356157/1128504153841336370/1200590104276783217
The text was updated successfully, but these errors were encountered: