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

resample methods in to_model_ds #341

Open
tomvansteijn opened this issue May 8, 2024 · 1 comment
Open

resample methods in to_model_ds #341

tomvansteijn opened this issue May 8, 2024 · 1 comment

Comments

@tomvansteijn
Copy link

When modelling a large area it can be necessary to coarsen the grid beyond the default 100 m resolution dictated by the REGIS dataset. The method to_model_ds uses resample.ds_to_structured_grid, using nearest neighbour interpolation by default (the argument is not set).

I think it would be useful to have other scaling methods for the hydraulic conductances, such as the mean and harmonic mean. Have you considered to implement these in nlmod?

@OnnoEbbens
Copy link
Collaborator

OnnoEbbens commented May 27, 2024

I agree that it would be nice to have this in nlmod. We have not yet implemented this. If you use the update_ds_from_layer_ds function there are more resampling methods than just nearest neighbor (example here: https://nlmod.readthedocs.io/en/stable/examples/12_layer_generation.html). However the harmonic mean is not yet an option in update_ds_from_layer_ds.

For geotop we use the arithmetic weighted mean for kh and a harmonic weighted mean for kv by default. So it would be nice to implement this logic for scaling in the update_ds_from_layer_ds function.

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

No branches or pull requests

2 participants