Hochfrequenz / ahbicht

Python Package that parses Condition Expressions from Anwendungshandbüchers ("AHB")
MIT License
4 stars 0 forks source link

Why do we raise an `RequirementEvaluationFailedBecauseOfUnknownNodesError` if the RC evaluation returns `UNKNOWN`? #275

Closed hf-kklein closed 1 year ago

hf-kklein commented 1 year ago

https://github.com/Hochfrequenz/ahbicht/blob/2802f7794c377dc2699a6840b86cee8412e676b5/src/ahbicht/expressions/requirement_constraint_expression_evaluation.py#L280

We should at least explain in a docstring. Exception basically means: Everything stops at this point, we cannot proceed in a meaningful way. I'd like to understand, why this is the case.

hf-kklein commented 1 year ago

@hf-aschloegl agrees: no error should be raised

hf-kklein commented 1 year ago

Ich bin heute wieder hierüber gestolpert, weil jemand anders auch danach gefragt hatte. Der Fehler (seit #321 ein spezifischer statt ein allgemeiner NotImplementedError) könnte ausgebaut werden aber das hat dann Auswirkungen: