Windows Utilities#861
Merged
Merged
Conversation
bd87dc5 to
917fad2
Compare
…utility downloads; updated build_examples to include powershell ps1 scripts
917fad2 to
9537145
Compare
Collaborator
|
@evan-greenbrg - could you give this a review when you get a chance? James thinks its ready to go! |
|
📊 Generated results:
URL: isofit/isofit-test-results#24 |
evan-greenbrg
approved these changes
Apr 9, 2026
| 2. Setup a Python environment | ||
| 3. Install ISOFIT and run examples as you would on Linux | ||
|
|
||
| ???+ info |
Collaborator
There was a problem hiding this comment.
Is the ???+ an mkdocs syntax? These show up when I render the .md via Gitub.
Collaborator
Author
Collaborator
|
@jammont This is great. My only question is posted above, but I'm guessing that this is mkdocs-specific formatting. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Recent troubles with Windows users:
gfortranis not installedbashis not installedIn an effort to ease these pain points, I've attempted the following:
data/cli/sixs.py. Sinceisofit download sixsalready supported a--mingwfor Windows which is used formaketo compile 6S, mingw also comes with gfortran. As such, I've set the precheck to prompt Windows users to install mingw via the CLI before 6S.data/cli/windows.pyto keep Windows utility installs together and separate from 6S. With this, I've added aisofit download windows --bashoption to install the portable version of Git Bash.isofit.inifile and inserted into the $PATH at runtime, so they're always availableisofit download allwill download these Windows utilities before anything else to ensure they're available for 6Sisofit dev bash, which will allow Windows users to follow the same docs instructions as Mac/Linux, hopefully simplifying a lot of headachesdata/build_examples.pynow generates.ps1scripts for Windows PowerShell as an additional option for usersCloses #847