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

Clean up the LPdiag tool #763

Open glatterf42 opened 7 months ago

glatterf42 commented 7 months ago

As described by @khaeru here, there are some small improvements to the LPdiag tool that we can make and should not forget about:

macflo8 commented 7 months ago

I would like to add a minor thing that I discovered yesterday:

The default value of the --up-tail click_option passed to the lp-diag tool is set to -5. The help string says that it is set to 5. It seems that the minus slipped in there.

https://github.com/iiasa/message_ix/blob/2d7b3f538d7a99edb8539fdc8831e5230e2ee23d/message_ix/tools/lp_diag/cli.py#L50

glatterf42 commented 7 months ago

Thanks for pointing this out!