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

Fixes/#116 entsoe py can not be used #117

Merged
merged 7 commits into from
Sep 19, 2023

Conversation

CarlosEpia
Copy link

@CarlosEpia CarlosEpia commented Aug 14, 2023

Fixes #116. This was already tested locally.
Each user must now have a hidden text file called ".entsoe-token" allocated in their home directory (in the same directory where the ERA5 password file is stored)

This branch can only be merged once #111

@CarlosEpia CarlosEpia self-assigned this Aug 14, 2023
@CarlosEpia CarlosEpia marked this pull request as ready for review August 14, 2023 10:07
Copy link

@ClaraBuettner ClaraBuettner left a comment

Choose a reason for hiding this comment

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

All GitHub checks fail, maybe because the setup file was not adjusted. Cpuld you please have a look at that?

Besides, it would be good to mention in the installation process which and where the file for the entsoe token should be added and how it could be created. We also once thought about using the old version with the downloaded files as a backup in case the direct access is not working.

@CarlosEpia CarlosEpia force-pushed the fixes/#116-entsoe-py-can-not-be-used branch from e63f1a4 to dce1800 Compare August 15, 2023 08:25
logging.warning(
"""Demand data from entsoe could not be retrieved.
Backup data is used instead"""
)
load_sq = pd.read_csv(
"data_bundle_powerd_data/entsoe/load_entsoe.csv", index_col="Index"
)

Choose a reason for hiding this comment

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

These lines now overwrite load_sq always, even if the data was retrieved by the website, right?

Copy link
Author

Choose a reason for hiding this comment

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

Right. Indentation error. Fixed in f618dad

@ClaraBuettner ClaraBuettner merged commit ee7657d into dev Sep 19, 2023
2 of 3 checks passed
@ClaraBuettner ClaraBuettner deleted the fixes/#116-entsoe-py-can-not-be-used branch September 19, 2023 09:21
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.

entsoe-py can not be used
2 participants