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.

Sign up for GitHub

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

Jump to bottom

Biomass transport #73

Merged
merged 31 commits into from
Sep 28, 2021
Merged

Biomass transport #73

merged 31 commits into from
Sep 28, 2021

Conversation

lisazeyen
Copy link
Collaborator

  • add option to split biomass potential to single nodes instead of global EU node
  • transport of solid biomass between nodes (costs based on JRC-EU-Times Bioenergy report and depend on country and distance)
  • add costs for upgrading biogas to gas
  • smaller changes:
    -> drop nan values when removing industry electricity demand to avoid warning (runtime error)
    -> add color for 'industry electricity' to color map in config.yaml
    -> move input data for transport to Snakefile

@lisazeyen
update mocksnakemake for testing
2b9ac51
@lisazeyen
move input data path for emobility/transport data to snakemake input
b4f1c57
@lisazeyen
Snakefile: add transport input data to rule prepare_sector_network
cdaa458
@lisazeyen
config: add missing color for industry electricity
50b4e82
@lisazeyen
Snakefile: add path to biomass transport cost in rule prepare_sector_…
2dd97c3 
…network
@lisazeyen
add biomass transport, split potential for solid biomass to nodes, ad…
651a7ff 
…d costs for upgrading biogas to gas
@lisazeyen
config.yaml: add color for biomass transport
3944e19
@lisazeyen
prepare_sector_network: split industry demand for biomass to single n…
ae9c0d9 
…odes
@lisazeyen
drop nan values to avoid runtime error when removing todays electrici…
13cae7d 
…ty demand
@lisazeyen
fix bug when adding biomass industry demand, add option to remove bio…
b4fbb47 
…mass transport and have only single EU biomass node
@lisazeyen
add option for solid biomass transport
4f2135a
@fneum fneum self-requested a review May 22, 2021 12:45
@fneum fneum mentioned this pull request May 25, 2021
Merged
@fneum
Copy link
Member

fneum commented Jul 2, 2021 •
edited
Loading

  • add tabula-py to environment (from conda-forge)
  • add "data/biomass/biomass potentials in europe_web rev.pdf" to databundle (via HTTP request)

@fneum fneum mentioned this pull request Jul 9, 2021
Merged
5 tasks
@fneum fneum added this to the Release 0.6 milestone Aug 3, 2021
lisazeyen
lisazeyen commented Sep 24, 2021
View reviewed changes
Copy link
Collaborator Author

@lisazeyen lisazeyen left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@fneum I only got one error when building the transport costs. I suggested a change and think this branch is ready to be merged.

scripts/build_biomass_transport_costs.py Outdated
# add missing Norway with data from Sweden
transport_costs["NO"] = transport_costs["SE"]

transport_costs.to_csv(snakemake.output.transport_costs)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I got an error here, I would suggest to change it to:
transport_costs.to_csv(snakemake.output[0])

@fneum fneum merged commit 7d0ff39 into master Sep 28, 2021
@fneum fneum mentioned this pull request Dec 4, 2021
Closed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fneum fneum fneum left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Release 0.6
Development

Successfully merging this pull request may close these issues.
2 participants
@lisazeyen @fneum