-
Notifications
You must be signed in to change notification settings - Fork 19
Open
Description
After installing (python 3.11, Windows 11) and running llms --serve 8000, I get:
C:\Users\Vlad>llms --serve 8000
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\Vlad\AppData\Roaming\Python\Python311\Scripts\llms.exe\__main__.py", line 6, in <module>
File "C:\Users\Vlad\AppData\Roaming\Python\Python311\site-packages\llms\main.py", line 4733, in main
init_extensions(parser)
File "C:\Users\Vlad\AppData\Roaming\Python\Python311\site-packages\llms\main.py", line 3401, in init_extensions
for item_path in get_extensions_dirs():
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\Vlad\AppData\Roaming\Python\Python311\site-packages\llms\main.py", line 3347, in get_extensions_dirs
extensions_path = get_extensions_path()
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\Vlad\AppData\Roaming\Python\Python311\site-packages\llms\main.py", line 3331, in get_extensions_path
return os.getenv("LLMS_EXTENSIONS_DIR", home_llms_path("extensions"))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\Vlad\AppData\Roaming\Python\Python311\site-packages\llms\main.py", line 2282, in home_llms_path
home_dir = os.getenv("LLMS_HOME", os.path.join(os.getenv("HOME"), ".llms"))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "<frozen ntpath>", line 108, in join
TypeError: expected str, bytes or os.PathLike object, not NoneType
The issue is that Windows doesn't have $HOME, my recommended fix would be to either use os.path.expanduser("~") or pathlib's Path.home().
Metadata
Metadata
Assignees
Labels
No labels