Closed amontoison closed 2 months ago
@tmigot We should probably add HS61, HS100 and HS219 in NLPModelsTest.jl
for more robust tests.
If we replace @NLconstraint
with @constraint
for the quadratic constraints in hs6
, hs10
, hs11
, and hs14
, almost all tests pass.
The failing tests are related to this issue: JuliaSmoothOptimizers/NLPModelsTest.jl#109.
Attention: Patch coverage is 74.27653%
with 80 lines
in your changes missing coverage. Please review.
Project coverage is 86.94%. Comparing base (
5bb0b82
) to head (02317b4
).:exclamation: Current head 02317b4 differs from pull request most recent head f39f1e4
Please upload reports for the commit f39f1e4 to get more accurate results.
Files | Patch % | Lines |
---|---|---|
src/utils.jl | 57.28% | 44 Missing :warning: |
src/moi_nls_model.jl | 65.04% | 36 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
close #50 close #149 close #150 @tmigot You should be happy now :smiley: