Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add more warning from QuadExpr #139

Merged
merged 1 commit into from
Sep 2, 2022

Conversation

amontoison
Copy link
Member

@amontoison amontoison commented Sep 1, 2022

@tmigot We forgot to add a warning for quadratic expressions (QuadExpr) if a model only has linear expressions (VariableRef or AffExpr).

@codecov
Copy link

codecov bot commented Sep 1, 2022

Codecov Report

Base: 91.98% // Head: 92.01% // Increases project coverage by +0.02% 🎉

Coverage data is based on head (c166a3a) compared to base (8468e63).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #139      +/-   ##
==========================================
+ Coverage   91.98%   92.01%   +0.02%     
==========================================
  Files           3        3              
  Lines         549      551       +2     
==========================================
+ Hits          505      507       +2     
  Misses         44       44              
Impacted Files Coverage Δ
src/utils.jl 91.76% <100.00%> (+0.06%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2022

Package name latest stable
NLSProblems.jl
OptimizationProblems.jl

@amontoison amontoison merged commit 02e6bc4 into JuliaSmoothOptimizers:main Sep 2, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant