SNEWS2 / snewpy

A Python package for working with supernova neutrinos
https://snewpy.readthedocs.io
BSD 3-Clause "New" or "Revised" License
26 stars 19 forks source link

Remove outdated integration test #207

Closed JostMigenda closed 1 year ago

JostMigenda commented 1 year ago

This fixes a minor oversight in #193 by removing a test script used to test the snewpy.snowglobes_interface.SNOwGLoBES class that was now removed. It also simplifies the GitHub integration test workflow.

github-actions[bot] commented 1 year ago

Result of Benchmark Tests

Benchmark Min Max Mean
python/snewpy/test/test_snowglobes.py::test_simulation_chain_benchmark 5.24 5.42 5.35 +- 0.08
mcolomermolla commented 1 year ago

After reviewing it, I don't see any reason why this PR shouldn't be merged. The changes made are straightforward after removing globes.

JostMigenda commented 1 year ago

Thanks for checking @mcolomermolla!