Skip to content

Update Configuration to include missing power model configs and sector-coupled configs#1842

Open
YuuGR1337 wants to merge 1 commit into
pypsa-meets-earth:mainfrom
YuuGR1337:fix/issue-1377
Open

Update Configuration to include missing power model configs and sector-coupled configs#1842
YuuGR1337 wants to merge 1 commit into
pypsa-meets-earth:mainfrom
YuuGR1337:fix/issue-1377

Conversation

@YuuGR1337

Copy link
Copy Markdown

Fixes #1377


@yerbol-akhmetov yerbol-akhmetov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi, @YuuGR1337. Thank you for the PR. Providing the details about what each configuration file does can be very useful. I have left few comments to be addressed.

Comment thread README.md

| Config file | Description |
|---|---|
| `config.default.yaml` | Default power model configuration with all available settings |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi, @YuuGR1337. Thank you for the PR. Can we modify the details a bit. config.default.yaml is primary baseline configuration for the full PyPSA-Earth workflow (electricity + sector options). We need to say that it serves as baseline and other config files inherent parameters from and override.

Comment thread README.md
| Config file | Description |
|---|---|
| `config.default.yaml` | Default power model configuration with all available settings |
| `config.tutorial.yaml` | Simplified configuration for test runs and tutorials |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we say that it is a lightweight tutorial override profile (tutorial: true) for short test/tutorial runs.

Comment thread README.md
Comment on lines +102 to +103
| `test/config.sector.yaml` | Sector-coupled model configuration for multi-energy system runs |
| `test/config.myopic.yaml` | Myopic optimization configuration |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe here it is better to refer: test/config.*.yaml, not individual configuration files inside test folder. These are the test configuration files that are used in CI test (run via make test) and also in some tutorials.

Comment thread README.md
Comment on lines +88 to +89
```
Remove the -n to do a real run. Follow the tutorial of PyPSA-Eur 1 and 2 on [YouTube](https://www.youtube.com/watch?v=ty47YU1_eeQ) to continue with an analysis.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These lines seems out of context, especially -n, as we do not talk about dry-run just before. Also, not sure about referring to PyPSA-Eur tutorials.

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.

Update Configuration to include missing power model configs and sector-coupled configs

2 participants