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

Enhance custom industry #302

Merged
merged 25 commits into from
May 2, 2024
Merged

Enhance custom industry #302

merged 25 commits into from
May 2, 2024

Conversation

hazemakhalek
Copy link
Collaborator

Changes proposed in this Pull Request

Better placement of custom data for industry with templates

Checklist

  • I tested my contribution locally and it seems to work fine.
  • Code and workflow changes are sufficiently documented.
  • Newly introduced dependencies are added to envs/environment.yaml and envs/environment.docs.yaml.
  • Changes in configuration options are added in all of config.default.yaml, config.tutorial.yaml, and test/config.test1.yaml.

Copy link
Collaborator

@Eddy-JV Eddy-JV left a comment

Choose a reason for hiding this comment

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

All is good except the comment I left in prepare sector network. In all cases we can imporve prepare_gas_network by making a PR in pypsa-earth to generate another GADM file for us instead of redoing aéverythig in prepare_gas_netowrk. But this can be done in another PR.

scripts/prepare_sector_network.py Show resolved Hide resolved
@hazemakhalek
Copy link
Collaborator Author

Approved by Eddy as communicated in a call

@hazemakhalek hazemakhalek merged commit 0901f59 into main May 2, 2024
5 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