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
RFC: Solution query docs #1731
RFC: Solution query docs #1731
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1731 +/- ##
=======================================
Coverage 68.69% 68.69%
=======================================
Files 30 30
Lines 3891 3891
=======================================
Hits 2673 2673
Misses 1218 1218 Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #1731 +/- ##
=======================================
Coverage 68.92% 68.92%
=======================================
Files 30 30
Lines 3923 3923
=======================================
Hits 2704 2704
Misses 1219 1219
Continue to review full report at Codecov.
|
docs/src/solutions.md
Outdated
|
||
```@docs | ||
JuMP.termination_status | ||
MOI.TerminationStatus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this display correctly? I didn't know you could display docstrings like this for other symbols in other packages.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't tested.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please build the docs locally to double check that they display as intended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@blegat, I understood that I cannot reference |
The missing reference for shape is fixed but there are other issues remaining: |
I am addressing these comments now. |
Sketch the solution query part of the manual
Targets: #1621