Replace Assertion that renewable resources exist in model regions, with checks + bypasses in build_renewable_profiles
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 148
- Forks
- 55
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 18
Description
This caused the following error to appear when running the default configuration file. I had to remove the `offwind` and `offwind_floating` techs to get it to work. If this is required, we should probably just remove them from the default config, or (ideally?)
Originally posted by @trevorb1 in https://github.com/PyPSA/pypsa-usa/pull/420#discussion_r1779057628
Trevor identified that we should ideally bypass a renewable resource rather than requiring the user to update the configuration file. This is for cases where a user is building a model of (example: Colorado, where Offshore Wind isn't eligible)
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate build_renewable_profiles and inspect how the default configuration handles offwind and offwind_floating when those resources are absent from model regions. Use the Colorado example and the error described in the issue to define the checks and bypass behavior. Done means users no longer need to remove ineligible technologies from the configuration.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100