Merge pull request #77 from lambda-feedback/tr112-atol-rtol-zero-behaviour

Fixed potential bug when either atol or rtol is zero but the other on…

Author: KarlLundengaard

app/evaluation.py

@@ -164,15 +164,15 @@ def replace_pi(expr):
                 return expr.subs(pi_symbol, float(pi))
             ans = replace_pi(ans)
             res = replace_pi(res)
-            if "atol" in params.keys():
+            if float(params.get("atol", 0)) > 0:
                 try:
                     absolute_error = abs(float(ans-res))
                     error_below_atol = bool(absolute_error < float(params["atol"]))
                 except TypeError:
                     error_below_atol = None
             else:
                 error_below_atol = True
-            if "rtol" in params.keys():
+            if float(params.get("rtol", 0)) > 0:
                 try:
                     relative_error = abs(float((ans-res)/ans))
                     error_below_rtol = bool(relative_error < float(params["rtol"]))