-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update doc #564
Update doc #564
Conversation
Codecov Report
@@ Coverage Diff @@
## master #564 +/- ##
==========================================
- Coverage 86.87% 86.39% -0.49%
==========================================
Files 47 47
Lines 4723 4718 -5
==========================================
- Hits 4103 4076 -27
- Misses 620 642 +22
Continue to review full report at Codecov.
|
* description of algorithms * solve lp ip form * fix BlockDecomposition docstrings * update doc ci * change julia version * ok * rm Manifest
println("Pattern of machine type $t used $nb_times_pattern_used times : $jobs_in_pattern"); | ||
end | ||
end | ||
``` |
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.
@laradicp it looks like there is a problem with the disaggregated solution interface, may you look what's going on ?
This is the bug from the documentation job log :
┌ Warning: failed to run `@example` block in src/man/decomposition.md:152-166
│ ```@example identical
│ for t in T
│ assignment_patterns = BD.getsolutions(model, t);
│ for pattern in assignment_patterns
│ nb_times_pattern_used = BD.value(pattern);
│ jobs_in_pattern = [];
│ for j in J
│ if BD.value(pattern, x[t, j]) ≈ 1
│ push!(jobs_in_pattern, j);
│ end
│ end
│ println("Pattern of machine type $t used $nb_times_pattern_used times : $jobs_in_pattern");
│ end
│ end
│ ```
│ c.value = BoundsError: attempt to access 1-element Vector{Coluna.ColunaBase.Solution{Coluna.MathProg.Formulation{Coluna.MathProg.DwMaster}, Coluna.MathProg.VarId, Float64}} at index [2]
└ @ Documenter.Expanders ~/.julia/packages/Documenter/R2HVS/src/Expanders.jl:591
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.
Yes, there's a problem with the usage of k
, it's not referring correctly to the subproblem. I'm working on it and I think the fix is just in getsolutions
. It worked with the test I made because the axis was unitary. Should I update the test too even if it takes more time to solve?
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.
No I think I can add a doctest
* Doc for artifical vars * file name * ok
Preview at https://atoptima.github.io/Coluna.jl/previews/PR564/
Literate
andKnapsackLib
more documentation about(2.3)specify
and multiplicityspecify
and multiplicity) example with Literate + use ofannotation
SolveLpForm
when the algorithm is only used to solve a linear problemChat with @laradicp :