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/#405 insert ch4 storages #472

Merged
merged 12 commits into from
Nov 12, 2021
Merged

Conversation

AmeliaNadal
Copy link
Contributor

Closes #405
This branch has been merged into the CI-branch.

gas_storages_list = gas_storages_list.reset_index(drop=True)

# Insert data to db
print(gas_storages_list)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this print statement necessary or can it be deleted?

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 deleted this print statement.

"""
Gas_grid_capacity = 130000 # G.Volk "Die Herauforderung an die Bundesnetzagentur die Energiewende zu meistern" Berlin, Dec 2012
N_ch4_nodes_G = ch4_nodes_number_G(define_gas_nodes_list()) # Number of nodes in Germany
print(N_ch4_nodes_G)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this print statement necessary or can it be deleted?

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 deleted this print statement.



def import_ch4_grid_capacity():
"""Define dataframe containing the gas storage modelling the ch4 grid storage capacity
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add some more information about the distribution of the stores/grid capacity here?

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 some more information about the modelling of the CH4 grid storage capacity (and open an issue to improve this modelling).

@ClaraBuettner
Copy link
Contributor

I run the code successfully and also had a look at the code. If you could add some more documentation about the comment (see above), I will approve this PR.

@AmeliaNadal AmeliaNadal merged commit d036c18 into dev Nov 12, 2021
@fwitte fwitte deleted the features/#405-insert-ch4-storages branch July 6, 2022 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate CH4 storages
3 participants