lanl-ansi / Juniper.jl

A JuMP-based Nonlinear Integer Program Solver
https://lanl-ansi.github.io/Juniper.jl/stable/
MIT License
180 stars 22 forks source link

Bugfix nlexpressions #185

Closed Wikunia closed 4 years ago

Wikunia commented 4 years ago

Copying the expression instead of mutating it in the expression graph. Fix #184

codecov[bot] commented 4 years ago

Codecov Report

Merging #185 into master will increase coverage by 0.01%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
+ Coverage   92.67%   92.68%   +0.01%     
==========================================
  Files          22       22              
  Lines        1993     1996       +3     
==========================================
+ Hits         1847     1850       +3     
  Misses        146      146              

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update fa8888e...843a572. Read the comment docs.

Wikunia commented 4 years ago

Does it look good @ccoffrin ? Additionally should this be a Bugfix for earlier versions as well?