Conversation
|
Pkg manager is failing: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #206 +/- ##
==========================================
+ Coverage 99.19% 99.38% +0.19%
==========================================
Files 5 9 +4
Lines 1487 1949 +462
==========================================
+ Hits 1475 1937 +462
Misses 12 12 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Thoughts on simplifying the documentation here: #207 We could move the PH tutorial to JuMP. I don't think POI is the right place for it. They're also not tested. |
|
How much of this was Claude? |
1 - I wrote a plan and the iterated with claude |
|
I am happy to move tutorials out of here |
|
best claude use case: handle ScalarQuadraticFunction's 2 versus 1/2 |
|
Okay. Rebase, delete the plan, and add some minimal docs. I've copy-pasted the PH tutorial; I'll add it to JuMP, and then we can switch it to POI once we tag the next release. |
|
Remove old quadratic stuff in a follow-up PR? |
|
Yeah a different PR. |
|
I rewrote the PH tutorial jump-dev/JuMP.jl#4117. Once we tag a new version of POI, we can change from Ipopt to HiGHS. |
|
I'm going to require 100% test coverage before we merge this. |
Clarify the conditions for cubic terms and remove redundant attention section.
Me too, I am working on it. |
|
@odow , this should be ready |
Closes #144
Should we remove
QuadraticObjectiveCoefin this PR?A: To do in a followup PR
Claude usage disclosure:
1 - I wrote a plan and iterated with claude on it.
2 - claude wrote a first pass.
3 - I have been cleaning up some mess for the last 2-days.