Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

add quadratic term in objective of conic term #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

matbesancon
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Mar 14, 2021

Codecov Report

Merging #27 (fe09980) into master (4652d04) will increase coverage by 0.08%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   85.80%   85.89%   +0.08%     
==========================================
  Files           5        5              
  Lines         317      319       +2     
==========================================
+ Hits          272      274       +2     
  Misses         45       45              
Impacted Files Coverage Δ
src/conic_form.jl 77.55% <100.00%> (+0.46%) ⬆️

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 4652d04...56808b1. Read the comment docs.

@matbesancon
Copy link
Collaborator Author

I added some tests but didn't get in MOI_copy_to where the objective is set, it is failing for now

@matbesancon
Copy link
Collaborator Author

ping @blegat on that. Not sure where in copy_to is the objective copied

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant