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

Issue #1161 metaswap regrid model #1168

Merged

Conversation

luitjansl
Copy link
Contributor

@luitjansl luitjansl commented Aug 21, 2024

Fixes #1161

Description

Implemented a regrid_like method for metaswap models

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

@luitjansl luitjansl marked this pull request as draft August 21, 2024 15:20
@luitjansl luitjansl marked this pull request as ready for review August 22, 2024 13:52
Copy link
Contributor

@JoerivanEngelen JoerivanEngelen left a comment

Choose a reason for hiding this comment

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

2 comments! Approved in advance

Comment on lines 261 to 262
mod2svat = self.pop("mod2svat", None)
grid = self.pop("grid", None)
Copy link
Contributor

Choose a reason for hiding this comment

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

These keys are not hardcoded, but decided by the user. I think you have to find packages based on their type and pop them I fear...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines 98 to 99
def is_regridding_supported(self) -> bool:
return False
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a comment that this can be changed when this is resolved #728

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link

@luitjansl luitjansl merged commit 4aef0c6 into metaswap_regridding_feature_branch Aug 23, 2024
6 checks passed
@luitjansl luitjansl deleted the issue_#1161_metaswap_regrid_model branch August 23, 2024 14:15
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