Change column sequence list to fix column identification bug

iiasa / message_ix

The integrated assessment and energy systems model MESSAGEix

https://docs.messageix.org

Apache License 2.0

112 stars 149 forks source link

Change column sequence list to fix column identification bug #762

Closed ywpratama closed 7 months ago

ywpratama commented 7 months ago

The bug was in line 208 of the lp_diag.py script. When generating self.mat DataFrame,"col":self.mat_row should be "col":self.mat_col to allow the tool correctly identify columns with bad coefficients

How to review

Read the diff and note that the CI checks all pass.

PR checklist

[ ] Continuous integration checks all ✅
[ ] Add or expand tests; coverage checks both ✅
[ ] Add, expand, or update documentation.
[ ] Update release notes.

CLAassistant commented 7 months ago

All committers have signed the CLA.