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

adapt venv_install.sh to install SMASH 1.0 with python-venv #173

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

maximejay
Copy link
Collaborator

adapt the bash script to install SMASH 1.0 into a virtual python environment using python-venv

@maximejay maximejay added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Apr 24, 2024
@maximejay
Copy link
Collaborator Author

Hello :)
J'ai juste adapté le script d'installe de SMASH via python-venv. J'ai viré l'installe de gdal... car si j'ai bien suivi, cette dépendance a été suprimée ? A priori pas besoin de python 3.11, 3.10 suffit (ubuntu 22.04).
Merci !!
Bonne journée
Max

@inoelloc
Copy link
Member

Hello :) J'ai juste adapté le script d'installe de SMASH via python-venv. J'ai viré l'installe de gdal... car si j'ai bien suivi, cette dépendance a été suprimée ? A priori pas besoin de python 3.11, 3.10 suffit (ubuntu 22.04). Merci !! Bonne journée Max

Salut, j'ai regardé rapidement le fichier, y a des choses que je ne comprends pas, notamment:

  • DEB_PACKAGE_NAME="gfortran build-essential gdal-bin libgdal-dev libshp-dev python3-gdal python3-venv python3-dev", tu laisses les lib gdal ?
  • pip install 'numpy>=1.13,<=1.23' pourquoi l'upper bound sur numpy ?
  • plus globalement sur les pip install, pourquoi ne pas faire directement pip install -r requirements-dev.txt ?

J'ai pas spécialement le temps d'itérer sur ce fichier mtn. Je te propose de le merge en l'état et de le modifier plus tard !

@inoelloc inoelloc self-requested a review April 30, 2024 08:52
@inoelloc inoelloc merged commit f902938 into maintenance/1.0.x Apr 30, 2024
8 checks passed
@inoelloc inoelloc deleted the venv_install branch April 30, 2024 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants