Closed zao111222333 closed 7 months ago
Attention: Patch coverage is 75.00000%
with 6 lines
in your changes are missing coverage. Please review.
Project coverage is 90.06%. Comparing base (
da57ff9
) to head (cd9e9c1
).
Files | Patch % | Lines |
---|---|---|
src/boolean_expression/_impl_boolean_expression.rs | 42.85% | 4 Missing :warning: |
src/boolean_expression/_impl_parser.rs | 88.23% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I'll just merge this as-is and fix the clippy warning on main, so that you don't have to wait for me to re-run the workflow :) Otherwise it looks good, thank you!
Released as 0.5.13
.
As https://github.com/sybila/biodivine-lib-bdd/issues/52, this PR supports
Conditional
ternary operator. Those features have been implemented:BooleanExpression
: Add fieldCond(Box<BooleanExpression>, Box<BooleanExpression>, Box<BooleanExpression>)
. And updateDisplay
forBooleanExpression
.parser
: Implement parser forCond
, with priority:imp
<cond
<or
.safe_eval_expression
: ImplementCond
'sBDD
with exsited functionfn if_then_else(a: &Bdd, b: &Bdd, c: &Bdd) -> Bdd
There is one potential todo validation:
BDD
equivalence ofA ? B : C
and(A & B) | (!A & C)