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

Add from_imod5_data method and test #1325

Conversation

JoerivanEngelen
Copy link
Contributor

@JoerivanEngelen JoerivanEngelen commented Dec 4, 2024

Fixes #1324

Description

Adds a ScalingFactors.from_imod5_data method. I think, the final missing constructor method.
Scaling factors for urban areas are set 1.0, not sure if iMOD5 does this as well @HendrikKok?

Checklist

  • Links to correct issue
  • Update changelog, if changes affect users
  • PR title starts with Issue #nr, e.g. Issue #737
  • Unit tests were added
  • If feature added: Added/extended example

Copy link

sonarcloud bot commented Dec 4, 2024

@JoerivanEngelen JoerivanEngelen merged commit 8e55dd5 into issue_#1260_from_imod5_data_metaswap Dec 5, 2024
6 of 7 checks passed
@JoerivanEngelen JoerivanEngelen deleted the issue_#1324_ScalingFactors_from_imod5_data branch December 5, 2024 10:50
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