NPV improvements results #998
Aurashk
started this conversation in
Show and tell
Replies: 1 comment 1 reply
-
|
Thanks sounds like this is working as expected. I am away at the moment but perhaps @tsmbland can take a look in the meantime - specifically are the values being produced for the metric as expected from manual calculation? |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I've implemented the suggestions for improving the npv calculation and for the 'simple' example it seems to be working more like you'd expect. copying the results here for a sanity check
input folder (enable broken results and make A0_RES use npv objective)
simple_test.zip
current output
simple_test_unchanged.zip
output when we add 1e-14 to each activity coefficient in npv objective (PR add epsilon term to npv activity coefficients #994)
simple_test_with_eps.zip
output when we remove AFC term from the optimisation (PR refactor and remove capacity term from npv objective #997)
simple_test_without_afc_term.zip
output when have both of the above changes (completes without errors!)
simple_test_both_changes.zip
Beta Was this translation helpful? Give feedback.
All reactions