Skip to content

Move shipping & aviation sector into separate function and config#1607

Merged
fneum merged 3 commits intomasterfrom
endogenous-shipping
Mar 24, 2025
Merged

Move shipping & aviation sector into separate function and config#1607
fneum merged 3 commits intomasterfrom
endogenous-shipping

Conversation

@fneum
Copy link
Copy Markdown
Member

@fneum fneum commented Mar 24, 2025

Changes proposed in this Pull Request

Separates functions add_aviation() and add_shipping() from add_industry().

This restructuring will simplify endogenizing the shipping and aviation sectors.

No other changes.

Checklist

  • I tested my contribution locally and it works as intended.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in config/config.default.yaml.
  • Changes in configuration options are documented in doc/configtables/*.csv.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • A release note doc/release_notes.rst is added.

@fneum fneum changed the title Move shipping sector into separate function and config Move shipping & aviation sector into separate function and config Mar 24, 2025
@fneum fneum force-pushed the endogenous-shipping branch from 25af4b1 to e652d5b Compare March 24, 2025 15:13
@fneum fneum marked this pull request as ready for review March 24, 2025 15:17
@fneum fneum merged commit 588fea1 into master Mar 24, 2025
9 of 11 checks passed
@fneum fneum deleted the endogenous-shipping branch March 24, 2025 15:30
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.

1 participant