Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Features/analyse results functions #615

Merged
merged 13 commits into from
Sep 26, 2023

Conversation

ClaraBuettner
Copy link
Contributor

I migrated some of the functions I wrote to analyze results to this eTraGo branch.
I also applied black, so it looks like some files changed a lot. But you could have a look at the other commits which do the real changes.

@CarlosEpia
Copy link
Contributor

I will review this PR, if it is ok to you @KathiEsterl

etrago/tools/network.py Outdated Show resolved Hide resolved
etrago/tools/calc_results.py Outdated Show resolved Hide resolved
@ClaraBuettner ClaraBuettner requested review from CarlosEpia and removed request for KathiEsterl March 29, 2023 08:47
Copy link
Contributor

@CarlosEpia CarlosEpia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some minor comments. After they are solved, this PR can be merged.

etrago/tools/calc_results.py Show resolved Hide resolved
etrago/tools/calc_results.py Show resolved Hide resolved
etrago/tools/plot.py Outdated Show resolved Hide resolved
Copy link
Contributor

@CarlosEpia CarlosEpia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from two minor comments, this branch can be merged into dev.

export_negative + import_positive - export_positive - import_negative
)

return marginal_cost, invest_cost, import_costs
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the function works well. Once system_costs_germany() is called, a tuple of 3 values is shown. Could you include in the output of this function what each of them represents?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The returned values are already described within the docstring. Do you mean that they should be described somewhere else?

@@ -3329,3 +3335,184 @@ def plot_heat_summary(self, t_resolution="20H", stacked=True, save_path=False):

if save_path:
plt.savefig(save_path, dpi=300)


def shifted_energy(self, carrier, buses):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since "self" was used as the first parameter, I tried to use the function typing etrago.shifted_energy(...), but it did not work. The function works well when it is imported directly from plot.py.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this function to the object in my latest commit

@ClaraBuettner ClaraBuettner merged commit 23e1da3 into dev Sep 26, 2023
6 checks passed
@ClaraBuettner ClaraBuettner deleted the features/analyse_results_functions branch September 26, 2023 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants