Closed trevorb1 closed 1 year ago
@HauHe
Hi @trevorb1, thanks for working on this so quickly. But I think here we had a little misunderstanding. What I meant in issue #46 is that when adding from the results parameter NewCapacity to the ResiudalCapacity of the next steps, only the NewCapacity for years within the step and not for all years in the model should be transfered, i.e., the first step is 5 years, which means the model is 10 years long, then only NewCapacity of the first 5 years should be passed to the ResidualCapacity of the next step, not of all 10 years. Does that make sense?
Hey @HauHe! Did you want to check the logic now to see if I got it right?
What I implemented is
I'll have a look here on GitHub now. And ask if I don't get it.
@trevorb1 you "spread" the NewCapacity over their life time in the function get_new_capacity_lifetime
in the main_utils.py
right?
And now the only the NewCapacity of the actual step years are forwarded, right?
I think it looks good, but haven't gone through each and every line.
I think we are on the same page, but you may want to review the logic just to make sure! See this code here:
Hi Trevor,
one potential issue that I see is the old_res_cap
. You take it from the data folder of the current step (both in the case that we have options and in the case that we don't), but shouldn't we in both cases take the ResidualCapacity of the next steps? and only the NewCapacity of the current step?
Ahh. good catch, maybe thats whats causing our issue!
I pushed a new commit to update the logic (however, please double check). However, we still have the issue of some of the scenarios failing when compared to the previous version :(
I have checked and the passing logic of the new capacity to the next steps seems to be correct now. Therefore, I merge it. Let's keep searching what's the issue.
Uups, wrong button at first ...
My bad for misunderstanding the residual capacity passing logic! I think this should address issue #46, as residual capacity is only added to the next step now, and not all subsequent steps.
We will need to update the flow chart representing this as well.
Closes #46