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

install Python only if not yet installed #235

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

Conversation

scriptPilot
Copy link

Closes #234

Root cause: Python might be installed already by Xcode developer tools or other sources before

@scriptPilot
Copy link
Author

It figures out it is even a bit easier.

Homebrew installes Xcode developer tools first which installs Python which installs Python TK.

So it must not be installed again in the install script :-)

@w8sl
Copy link

w8sl commented Mar 17, 2024

"brew install gdal" will install Python 3.12 anyway, as dependency. scikit-fmm for Python 3.12 has not been relased yet. Therefore, it is necessary to force the whole install into python3.11 (or 3.10 if preffered). See comment:
#210 (comment)

@scriptPilot
Copy link
Author

Ok, well I will try the linked steps by step - thanks for that.

I think it would be really helpful to have the same Docker image for all platforms in the future.

Only Docker will be required - and maybe XQwartz for MacOS (tutorial).

@w8sl
Copy link

w8sl commented Mar 18, 2024

I am trying to update install instructions for 1.40 on my fork

@scriptPilot
Copy link
Author

My attempt to install instructions. I don't know how to install scikit-fmm on Arch. Install_Instructions.txt install_mac.sh.txt requirements.txt

Would be helpful to have the installation instructions per OS and later for Docker in the README file.
I spent a couple of hours to try on MacOS as the good instruction wasn't found before 😬

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.

Error: This environment is externally managed
2 participants