Skip to content
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

rm support of anonymous variables & constraints #565

Merged
merged 1 commit into from
Jul 21, 2021
Merged

Conversation

guimarqu
Copy link
Contributor

@guimarqu guimarqu commented Jul 20, 2021

fix #553

At the moment, Coluna does not detect anonymous variables when direct_mode = false but the annotation of an anony var is nothing (cf atoptima/BlockDecomposition.jl@d86fa7e)

@guimarqu guimarqu requested a review from laradicp July 20, 2021 19:27
@codecov
Copy link

codecov bot commented Jul 20, 2021

Codecov Report

Merging #565 (8ee8578) into master (24ec00b) will increase coverage by 0.07%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #565      +/-   ##
==========================================
+ Coverage   86.15%   86.22%   +0.07%     
==========================================
  Files          47       47              
  Lines        4687     4698      +11     
==========================================
+ Hits         4038     4051      +13     
+ Misses        649      647       -2     
Impacted Files Coverage Δ
src/decomposition.jl 99.19% <100.00%> (+0.04%) ⬆️
src/MathProg/vcids.jl 72.41% <0.00%> (+5.74%) ⬆️

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 24ec00b...8ee8578. Read the comment docs.


m = BlockModel(coluna)
anonymous_var_model!(m)
# The variable is annotated in the master.
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't understand why this happens.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Neither do I

Copy link
Contributor

Choose a reason for hiding this comment

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

Haha ok. I tried to find a reason but couldn't yet, I can keep searching if it's relevant.

@guimarqu guimarqu merged commit d5fd81d into master Jul 21, 2021
@guimarqu guimarqu deleted the forb_anonym branch July 26, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support of JuMP anonymous constraints/variables
2 participants