Closed SMoraisAnsys closed 5 days ago
Thanks for opening a Pull Request. If you want to perform a review write a comment saying:
@ansys-reviewer-bot review
Attention: Patch coverage is 33.33333%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 83.15%. Comparing base (
df9b411
) to head (c993a1b
). Report is 2 commits behind head on main.
As title says.
Note: contrary to previous refactoring, I used the following approach to be more compliant with what was expected to happen:
AssertionError
when calling assert in the code.The first ensures that we can log information and don't confuse users. The second ensures that the tolerant approach used through the function handler is kept and used.