-
Notifications
You must be signed in to change notification settings - Fork 37
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
System cannot find the path specified #58
Comments
Cc @@concretevitamin @Michaelvll, this looks like it might be an issue in the SkyPilot launch flow on Azure, any ideas? |
It may be related to Windows paths. Not sure if we tested Windows. Cc @romilbhardwaj |
We haven't tested running natively on Windows. However, I can confirm that SkyPilot works in a python environment inside Windows Subsystem for Linux (WSL). Perhaps windows users can use WSL as a workaround? |
@romilbhardwaj I did some more looking into the code and found that one of the issues appears to be from the temp yml file being generated that specifies the properties of the machine. file_mounts: {
"~/.sky/sky_ray.yml": "C:\Users\stollbak\.sky\generated\test.yml.tmp",
"~/.sky/wheels/5f10c8a76630d0f617f0312055a347bf": "C:\Users\stollbak\AppData\Local\Temp\5f10c8a76630d0f617f0312055a347bf",
"~/.azure/azureProfile.json": "~/.azure/azureProfile.json",
"~/.azure/clouds.config": "~/.azure/clouds.config",
"~/.azure/config": "~/.azure/config",
"~/.azure/msal_token_cache.json": "~/.azure/msal_token_cache.json",
} Looks like backslashes are creating issues since they are being treated as escape characters. Where is this file being generated in the library? I can take a look and see about a fix. |
Hi @BStoller, the tmp yml file is generated here. This method fills in a template YAML. The offending paths in your example above are called
We're very open to contributions and I look forward to your PR in the SkyPilot repo :) |
@BStoller , I'm happy to help write up a change if you'd test them on your windows box. I actually have a few other changes I was thinking to contribute up to SkyPilot shortly. |
@dongreenberg No problem, I can test the changes. Let me know if you need any more information. |
This is problem from os.devnull usage (in skypilot) in windows. Windows does not offer os.devnull at default, but WSL offers it. Thus the result is different. import os
log_path = os.devnull
log_dir = os.path.expanduser(os.path.dirname(log_path)) This will result '' for some windows environment, but will work in linux or its subsystems. It is headache for those windows systems- windows can't do that. |
Describe the bug
I'm having an issue when trying to start up a Lang chain llm. After setting up the cluster
I attempt to create the llm that will run my inferences
My code appears to run into some error with creating / finding a file. Hoping you all would be able to support.
Versions
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: