search

recast-hep / recast-atlas

CLI for ATLAS RECAST contributors
https://recast.docs.cern.ch/
Apache License 2.0
7 stars 5 forks source link

ci: Fix Python tests CI workflow #43

Closed matthewfeickert closed 3 years ago

matthewfeickert commented 3 years ago

Requires PR #44 to go in first.

The CI is failing as the default database cache that the base images have are out of sync with the package lists for Ubuntu and also because of conflicts with yadage[viz]. Running apt-get update first will retrieve the latest package lists and ensure that apt-get install will proceed. Also manually install the components of yadage[viz]. Additionally update the build status badge to reflect the state of GitHub Actions on master.

CI

matthewfeickert commented 3 years ago

Ah. I now see that given

https://github.com/recast-hep/recast-atlas/blob/f109a197b1a01b08fe9f0ad75fc0967210772c80/setup.py#L33-L39

there are indeed some difficulties with getting

recast run testing/busyboxtest --backend local

to work. :slightly_smiling_face:

matthewfeickert commented 3 years ago

Okay I have pytest passing now, but I'm going to open up PR #44 that will split out all the CIs into separate workflows and then have this one refocus on just cleaning up the Python tests and then we can clean up the linting and everything else in separate PRs.

matthewfeickert commented 3 years ago

@lukasheinrich This is ready for review. It only fixes the Python CI and the Linting and Docker CI will get fixed in follow up PRs.