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.

Sign up for GitHub

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

Jump to bottom

Format failing on master #2673

Closed
odow opened this issue Aug 24, 2021 · 4 comments · Fixed by #2676
Closed

Format failing on master #2673

odow opened this issue Aug 24, 2021 · 4 comments · Fixed by #2676
Assignees
@odow
Labels
Type: Bug

Comments

@odow
Copy link
Member

odow commented Aug 24, 2021

#2672 (comment)
@odow odow added the Type: Bug label Aug 24, 2021
@odow odow self-assigned this Aug 24, 2021
@pulsipher
Copy link
Contributor

If it helps, I found that the error goes away if I comment out the following lines:

JuMP.jl/src/lp_sensitivity.jl

Lines 393 to 395 in 2e23b5e

ej = zeros(size(A, 1))
ej[findfirst(isequal(var), vars[basic[1:length(vars)]])] = 1.0
N_red = A[:, .!basic]' * (A[:, basic]' \ ej)

I haven't figured out why though.

@odow
Copy link
Member Author

odow commented Aug 24, 2021

I found the culprit: JuliaLang/Tokenize.jl@v0.5.18...v0.5.19

Tokenize changed.

@odow odow mentioned this issue Aug 24, 2021
Closed
@odow
Copy link
Member Author

odow commented Aug 24, 2021

Upstream issue: JuliaLang/Tokenize.jl#189

@odow
Copy link
Member Author

odow commented Aug 25, 2021

Temporarily fixed by #2674, but we should await an upstream resolution.

@odow odow mentioned this issue Aug 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@odow odow

Labels
Type: Bug
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Revert Tokenize version fix jump-dev/JuMP.jl
2 participants
@odow @pulsipher