Skip to content
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

Now Windows-compatible #61

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Now Windows-compatible #61

wants to merge 17 commits into from

Conversation

lopez-brau
Copy link

  • Updated most of the functions in rwebppl.R to be Windows-compatible by including conditionals based on the user's OS.
  • Added a new folder under inst named powershell. This folder contains installation scripts for Windows users that are the equivalent to those found in inst/bash.

The only feature that is currently still not supported is installing via GitHub since Windows doesn't natively support git.

@lopez-brau
Copy link
Author

Previously, install-dev-webppl.sh didn't have a Windows equivalent because it installed WebPPL via git. I made a Windows-equivalent (install-dev-webppl.ps1 within inst/powershell) that installs WebPPL via npm. Windows users should now be able to install RWebPPL using the same command as Linux and Mac users.

@lopez-brau
Copy link
Author

Verified that this fix also works for Windows 11. Updated README to reflect that RWebPPL is operational on Windows, and included git as an additional system requirement for Windows users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant