OFR-IIASA

@behnam-zakeri it seems this PR was automatically closed as the branch to which this PR was originally made has been deleted. Can you reopen this PR and change the destination...

@behnam-zakeri thanks for the comments; these have been addressed. I am not planning to add tests. As we generally dont test the GAMS-core formulation, doing so for this feature would...

@gidden there is unfortunately no off the shelf example to which to test this change i.e. we have nothing of the shelf to test change in period duration. This would...

@gidden and @behnam-zakeri I have managed to extend the `make_westeros()` testing utility to accept variants of the `model_horizon`. This seems to suite our requirements in terms of testing the changes...

@behnam-zakeri as discussed yesterday, let us extend the calculation of the levelized costs to include the relation-related costs as well. Also, the issues raised on @khaeru can also be addressed...

@gidden I agree, and also to test the 'var_cost' as well as the additions to the formulation related to relations. I believe by solving using the command solve(var_list=["levelized_cost"]) we will...

In order to address the issues associated with creating tests for this PR, an issue has been started, see #661 Once the issues have been resolved and the parameter can...

@behnam-zakeri , my proposal would be to merge this PR first and then change the PR for #658 to main. The reason is that while here we are only just...

I can confirm the issue. I have tried to do the same and encountered that case (1) as described above doesnt. In addition I think is also worth mentioning that...

Correct. Using the `editable` option, all tests pass except for `ixmp/tests/test_tutorials.py`, which yields the following error: ``` > raise CellExecutionError.from_cell_and_msg(cell, exec_reply_content) E nbclient.exceptions.CellExecutionError: An error occurred while executing the following...