-
Notifications
You must be signed in to change notification settings - Fork 92
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
Windows Support #31
Comments
Is it difficult to support? I just got a Windows laptop with 4060 (8GB VRAM) GPU last night, and have successfully run 4bit-quantized Mistral 7B via https://github.com/oobabooga/text-generation-webui#one-click-installers and also via https://github.com/LostRuins/koboldcpp#windows-usage, so I can certainly help testing. |
Currently, the main non-Windows-compatible part is when we want a system function that's not supported by Lean (e.g., to count the number of CPUs, or check if the system is ARM64 or X64), we use some Unix-specific workaround (e.g., Thanks for offering to test it. We'll work on it and post updates here. |
BTW, Windows with WSL has already been tested to work. |
I managed to get Copilot working on Windows (WSL) after about two hours of effort, in large part because I missed the very important acronym "WSL" in the list of requirements. My workflow ended up being as follows:
I think it would help users who are not already experienced in installing open source software to (a) clarify exactly how to add a package configuration option to lakefile.lean (viz., by placing it inside the "package {...}" fragment of that lean file), and (b) emphasizing that the software does not directly run in Windows itself (it took me about an hour of trying to "debug" the lean file (which was not reporting a "run_io" feature) until finally hitting the "panic! "Windows is not supported"" error message). |
Hi @teorth, Thank you for your valuable suggestions. I have incorporated them into the current README. LeanCopilot doesn't support native Windows because Lean didn't have cross-platform implementations for some features we need, so we resorted to workarounds that are Unix-specific. However, maybe that's no longer the case with the most recent version of Lean. If so, it may make sense to support Windows directly. @Peiyang-Song Can you take a look when you get a chance? |
I'm not sure how important it is to support Windows (how many Lean users actually use Windows?)
The text was updated successfully, but these errors were encountered: