Skip to content

Conversation

@michaelbynum
Copy link
Contributor

Changes proposed in this PR:

  • Add a test for performing FBBT with named expressions
  • Improve error messages in FBBT when a constraint is found to be infeasible

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the BSD license.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@codecov
Copy link

codecov bot commented May 11, 2022

Codecov Report

Merging #2392 (a39069c) into main (762d6ca) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2392   +/-   ##
=======================================
  Coverage   85.86%   85.86%           
=======================================
  Files         617      617           
  Lines       76163    76163           
=======================================
+ Hits        65396    65400    +4     
+ Misses      10767    10763    -4     
Flag Coverage Δ
linux 82.60% <ø> (+<0.01%) ⬆️
osx 72.83% <ø> (ø)
other 82.79% <ø> (ø)
win 79.65% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pyomo/contrib/mindtpy/iterate.py 83.56% <0.00%> (+1.82%) ⬆️

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 762d6ca...a39069c. Read the comment docs.

@jsiirola jsiirola merged commit 550b576 into Pyomo:main May 13, 2022
@michaelbynum michaelbynum deleted the appsi_fbbt branch July 6, 2022 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants