search

ImperialCollegeLondon / sharpy

Simulation of High Aspect Ratio aeroplanes and wind turbines in Python: a nonlinear aeroelastic code
https://imperial.ac.uk/aeroelastics/sharpy
BSD 3-Clause "New" or "Revised" License
123 stars 58 forks source link

Added pip install (with docs) #280

Closed ben-l-p closed 7 months ago

ben-l-p commented 8 months ago

Extension of Alvaro's pull request HERE, with added documentation and changes to the Github workflows. The Docker build still uses Conda/Mamba - ideally I want to remove this dependency.

Also fixed SciPy issues, as it was using a depreciated FFT function

codecov-commenter commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.69%. Comparing base (ee7206f) to head (906a566). Report is 24 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #280 +/- ## ======================================= Coverage 67.69% 67.69% ======================================= Files 166 166 Lines 26709 26709 ======================================= Hits 18081 18081 Misses 8628 8628 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ben-l-p commented 7 months ago

Hold off on approving - some packages for a install have been forgotten

ben-l-p commented 7 months ago

Changes have been made to the Mayavi install (required for TVTK) as there is a bug which can causes installation to fail with pip. It now git clones the latest version for installation - this has been tested on Python 3.10/3.11 on both Ubuntu and Mac.

Also added a second Conda environment for Apple Silicon, as Conda is yet to fully include GCC/GFortran for this. This is purely removing these packages so that it will still "work", you just need to install them yourself instead.

Should all be working now - please test!