-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Making the code platform-agnostic would lead to more contributions #209
Comments
+1 |
I think it supports all OS and more importantly all versions of windows. |
Just install WSL if you're on windows, it should work flawlessly |
Installing WSL should do the trick |
I understand that WSL is a solution. My concern is that WSL may be difficult to install for some. I also recognize that Unix-based OS is also difficult for some to learn as well. The aim is to make it more platform agnostic: Using Python libraries will achieve this aim and facilitate more contributions. |
Oh, yeah... I totally understand your view now.
I hope it's implemented soon
…On Thu, Mar 16, 2023, 12:06 PM jonathanagustin ***@***.***> wrote:
@Toby-Mascot <https://github.com/Toby-Mascot> @SinanAkkoyun
<https://github.com/SinanAkkoyun>
I understand that WSL is a solution. My concern is that WSL may be
difficult to install for some. I also recognize that Unix-based OS is also
difficult for some to learn as well. The aim is to make it more platform
agnostic: Using Python libraries will achieve this aim and facilitate more
contributions.
—
Reply to this email directly, view it on GitHub
<#209 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASXGAFIR7443YSNZGLCAWMLW4LX4LANCNFSM6AAAAAAV4S4BUU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Most of the dev team at OpenAI uses MacOS, but you are welcome to open a PR to add windows related info! |
I observed that the example Jupyter notebooks contain OS-specific code. For example, in
evals/examples/lafand-mt.ipynb
, there is an assumption of a Unix filesystem:Here, it seems better to use Python's
tempfile
module to handle temporary files and directories across different platforms.In
evals/examples/mmlu.ipynb
, there are these commands:Here, it seems better to use a Python library like
urllib
to download datasets because it is built into the language and is usable across different operating systems.Multi-platform support would lead to more contributions. Instead of using Unix-specific methods to handle the filesystem, Python libraries can be used instead. Using a Python library would generally handle most of the OS-specific issues.
I do not have any specific problems, but I recognize that others may not have access to different operating systems. Perhaps they may not be technically proficient enough to use WSL or Docker.
The text was updated successfully, but these errors were encountered: