-
Notifications
You must be signed in to change notification settings - Fork 70
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
Update Github Actions #461
Conversation
sudo apt-get update | ||
sudo apt-get install libmuparser-dev libhdf5-serial-dev libomp5 libomp-dev libfftw3-dev libcfitsio-dev lcov doxygen graphviz | ||
sudo apt-get install pandoc # do not only use pip to install pandoc, see https://stackoverflow.com/questions/62398231/building-docs-fails-due-to-missing-pandoc | ||
pip install -r doc/pages/example_notebooks/requirements.txt # load requrements for notebooks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
requirements
Hi @JulienDoerner |
hey @lukasmerten, |
If re-running the test fixes the problem, which seems to be the case, we don't need to exclude the test. I will take a closer look at the doc and coverage action in the next days. |
From my point of view we can merge this, when the typo is fixed. I'll approve it now, but let's wait another day or so for additional comments. |
Dear all,
this PR adds two new actions.
crpropa-testing
test (Ubuntu 20, python3). The notebooks are converted to python scripts and executed afterwards. This test aims to check if interfaces are changed and whether the documentation is updated accordingly.At the moment this test excludes the notebooks:
custom-photon-field
,MHD_models
,density_grid_sampling
,lensing_cr
andlensing_map
. Those notebooks require external data, which are not provided within this repository.Also in the notebooks the DINT/ELECA part is removed and some ploting is updated to newer matplotlib versions.