Skip to content

Conversation

yechenyan
Copy link
Contributor

@yechenyan yechenyan commented Feb 25, 2025

…and ensure output directory exists

Issue

I did not modify the configuration and ran the simulation using snakemake -call all --configfile=config/config.public.yaml. The following error was encountered:
image

The cause is as follows:

  1. Since this is my first time running it, there is no data/egon folder, which will cause a wget -o error.
  2. macOS and Windows systems do not have wget.

Solution

  1. Create data/egon before downloading files.
  2. Use curl to download files.

Before asking for a review for this PR make sure to complete the following checklist:

  • Workflow with target rule ariadne_all completes without errors
  • The logic of export_ariadne_variables has been adapted to the changes
  • One or several figures that validate the changes in the PR have been posted as a comment
  • A brief description of the changes has been added to Changelog.md
  • The latest main has been merged into the PR
  • The config has a new prefix of the format YYYYMMDDdescriptive_title

Copy link
Member

@lkstrp lkstrp left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix @yechenyan!

In general, wget is preferred over curl for downloading large files. On MacOS you can also just do brew install wget. But since these are not large files, it's fine. I am also not even sure why this was implemented via bash in the first place.

cpschau pushed a commit that referenced this pull request Mar 20, 2025
adapt co2 targets to scenario
@lkstrp lkstrp merged commit 281c32a into PyPSA:main Apr 4, 2025
4 of 9 checks passed
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.

2 participants