Skip to content

Conversation

@odow
Copy link
Member

@odow odow commented Dec 4, 2020

Test will fail until #161 is merged, so this is just a draft.

Closes #160

Also address #146

@odow odow marked this pull request as ready for review December 4, 2020 02:27
@odow odow closed this Dec 4, 2020
@odow odow reopened this Dec 4, 2020
@codecov
Copy link

codecov bot commented Dec 4, 2020

Codecov Report

Merging #162 (59265eb) into master (6c8fb69) will increase coverage by 0.03%.
The diff coverage is 86.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   84.39%   84.43%   +0.03%     
==========================================
  Files           8        8              
  Lines        1474     1497      +23     
==========================================
+ Hits         1244     1264      +20     
- Misses        230      233       +3     
Impacted Files Coverage Δ
src/MOI_wrapper/MOI_callbacks.jl 96.47% <86.36%> (-3.53%) ⬇️
src/MOI_wrapper/infeasibility_certificates.jl 91.66% <0.00%> (+0.17%) ⬆️

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 6c8fb69...59265eb. Read the comment docs.

@odow odow merged commit a901911 into master Dec 6, 2020
@odow odow deleted the od/cb_node_status branch December 6, 2020 21:41
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.

Implement MOI.CallbackNodeStatus

2 participants