Skip to content

Added @retry decorator with default values

Luis Aleixo requested to merge feature/pipeline_retry into master

This MR solves the latest error on the pipeline.

After a discussion with @nimounet, we concluded that whenever there is a change in the tests that happen before (chronologically) the Monte-Carlo tests (which is the case of the latest change on the short_range_distances - https://gitlab.cern.ch/cara/cara/-/merge_requests/346, then it might affect the random samples (which are seeded at the beginning of all the tests.

This way, some tests might fail, which is the case of the test_monte_carlo_full_models.py. To avoid these errors we could play with the SAMPLE_SIZE and tolerance, but instead we implemented one solution with the @retry decorator with default parameters.

Useful links:

Merge request reports