Skip to content

Conversation

mlubin
Copy link
Member

@mlubin mlubin commented Jul 5, 2019

Closes #766

@mlubin mlubin requested a review from blegat July 5, 2019 19:29
@codecov-io
Copy link

codecov-io commented Jul 5, 2019

Codecov Report

Merging #781 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #781   +/-   ##
=======================================
  Coverage   94.26%   94.26%           
=======================================
  Files          59       59           
  Lines        6481     6481           
=======================================
  Hits         6109     6109           
  Misses        372      372
Impacted Files Coverage Δ
src/Utilities/model.jl 91.46% <ø> (ø) ⬆️
src/sets.jl 92.72% <ø> (ø) ⬆️

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 719bc68...9ab3e88. Read the comment docs.

(MOI.VectorOfVariables,),
(MOI.VectorAffineFunction, MOI.VectorQuadraticFunction))

@doc raw"""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one, I was wondering how we could document that struct that is creating inside a macro

@mlubin mlubin merged commit bd88e5d into master Jul 6, 2019
@mlubin mlubin deleted the ml/model branch July 6, 2019 13: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.

Add default instantiation of a Model
3 participants