Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.85%. Comparing base (6e72af3) to head (5cee0bc).
Report is 7 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1603      +/-   ##
==========================================
- Coverage   76.85%   76.85%   -0.01%     
==========================================
  Files         394      394              
  Lines       63241    63244       +3     
  Branches    10359    10359              
==========================================
  Hits        48604    48604              
- Misses      12188    12192       +4     
+ Partials     2449     2448       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Could we augment this function with an additional function that is a drop-in replacement for expr_if? Maybe have two expr_if replacements, depending on whether the underlying expression is continuous (when we can use smooth_max or smooth_min or discontinuous, when we use smooth_heaviside.

After thinking about this a bit more (and trying to implement it), I'm finding it difficult to cover all the possible scenarios since there's not necessarily uniformity in how the conditional statement is set up (number of terms on the LHS and RHS). If we do assume a simple if x > y or x < y, I feel that it is fairly straightforward for the users to replace the Expr_if themselves. That way we don't need to keep Expr_ifs in the code at all (i.e. "the drop-in replacement" function would need to be passed the Expr_if)