Skip to content

Pass geotop layers to get_combined_layer_models#414

Merged
dbrakenhoff merged 1 commit intodevfrom
pass_geotop_layers
Mar 14, 2025
Merged

Pass geotop layers to get_combined_layer_models#414
dbrakenhoff merged 1 commit intodevfrom
pass_geotop_layers

Conversation

@dbrakenhoff
Copy link
Collaborator

Allows for using different option for parsing paleochannels without having to run all methods separately.

Usage:

geotop = nlmod.read.geotop.get_geotop(extent)
gt_layered = nlmod.read.geotop.to_model_layers(geotop, method_geulen="add_to_layer_above")

nlmod.read.regis.get_combined_layer_model(extent, use_regis=True, use_geotop=True, gt_layered=gt_layered)

- allows for using different option for parsing paleochannels without having to run all methods separately.
@dbrakenhoff dbrakenhoff added the enhancement New feature or request label Feb 28, 2025
@dbrakenhoff dbrakenhoff self-assigned this Feb 28, 2025
@github-project-automation github-project-automation bot moved this to Todo in NHFLO Feb 28, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in NHFLO Mar 14, 2025
@rubencalje
Copy link
Collaborator

Does this have anything to do with issue #420?

@dbrakenhoff
Copy link
Collaborator Author

Does this have anything to do with issue #420?

Indeed it does :)

@dbrakenhoff dbrakenhoff merged commit 9d2b421 into dev Mar 14, 2025
2 of 3 checks passed
@dbrakenhoff dbrakenhoff deleted the pass_geotop_layers branch March 14, 2025 11:24
@github-project-automation github-project-automation bot moved this from In Progress to Done in NHFLO Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants