Features/#342 create scenations one pipeline #345
Conversation
…year for the scenario path
| @@ -198,7 +198,8 @@ def load_biogas_generators(scn_name): | |||
| ) | |||
| target_file = Path(".") / "datasets" / "gas_data" / basename | |||
|
|
|||
| urlretrieve(url, target_file) | |||
| if not target_file.is_file(): | |||
There was a problem hiding this comment.
In our current setting, that would mean we should manually copy-paste the file before this task is executed (since the file is not online anymore), right?
There was a problem hiding this comment.
If the file is there, the task does not try to download it anymore. In case the file is not there, it still tries to download it. That way, I could use the copy-pasted file without any errors.
There was a problem hiding this comment.
I think they replaced the file with a newer version (see #337 (comment))
| ) | ||
|
|
||
| if target.empty: | ||
| print(f"No PV rooftop in scenario {scenario}") |
There was a problem hiding this comment.
I think this is necessary because we are getting a unique solar carrier for both solar ground-mounted and rooftop. Do you know if that will be corrected in the next models coming from Pypsa-eur?
There was a problem hiding this comment.
I think this should be fixed by pypsa-eur. But we could still keep the check in there, right?
ClaraBuettner
deleted the
features/#342-create-scenations-one-pipeline
branch
Deals with #342
It was running successfully for Schleswig-Holstein