cta-observatory / ctapipe

Low-level data processing pipeline software for CTAO or similar arrays of Imaging Atmospheric Cherenkov Telescopes

https://ctapipe.readthedocs.org

BSD 3-Clause "New" or "Revised" License

62 stars 265 forks source link

Fix CI #2547

Closed maxnoe closed 2 months ago

maxnoe commented 2 months ago

Fix CI by slightly adapting impact test to find a minimum on both macos and linux... not a great fix, but the test needs to be improved some other way.

The new pytest version prints out the full stack trace for xfails in the CI log, which makes it harder to spot the actual error that caused the test failure.

Checking the xfails, we only have these once here, which are not really xfails but really expected, so I removed the xfail.

maxnoe commented 2 months ago

The macos-13 / macos-14 tests are running much faster now than the old macos-latest (macos 12) tests. Nice! thanks github