FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add processing for shipping routes by irm-codebase · Pull Request #21 · modelblocks-org/module_area_potentials · GitHub

Add processing for shipping routes - #21

Open
irm-codebase wants to merge 5 commits into
feature/improved-clippingfrom
feature/ship-traffic
Open

Add processing for shipping routes#21
irm-codebase wants to merge 5 commits into
feature/improved-clippingfrom
feature/ship-traffic

Conversation

irm-codebase commented Aug 13, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes #20

Targets PR #18. Kept separate to make reviews easier.

Summary of changes in this pull request

  • Adds processing for the World Bank's Global Shipping traffic Density dataset
  • Download and use is optional: the unzipped file is 9 GB, so I made use of snakemake's branch functionality to only use it when requested in the config.
  • Default is set to match PyPSA-Eur's default of 400(AIS) * 8760(h) * 6(yr) = 2.1e7
    The dataset has cumulative counts of hourly AIS positions (a ship saying "I'm here", per hour) over 6 years.
    It can be seen as roughly "there was an average of 400 ships at this spot per hour between 2015 and 2021".
    The value seems to have been tuned to exclude ports and heavy-traffic routes.

Resampled example for Italy

Example of resulting file for the Netherlands, keeping other defaults the same

Reviewer checklist

  • There are no pip dependencies in the module's environment files (workflow/envs/).
  • All rules use pathvars (e.g., <results>) in their inputs and outputs.
  • The integration test-suite is successful, including:
    • pre-commit.ci tests pass.
    • tests pass for all relevant OS configurations (linux, osx, windows).
  • Module documentation is up-to-date, including:
    • INTERFACE.yaml mentions all relevant pathvars and wildcards.
    • README.md describes how to use the module and has the necessary citations.

jnnr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks good! I will test this soon - let me know if I should do this before you merge.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

Add exclusion for marine zones with high traffic

2 participants


Back | FazBrowse Home | New Git URL