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 information about geotop stratigraphic units #361

Merged
merged 4 commits into from
Aug 27, 2024

Conversation

rubencalje
Copy link
Collaborator

This PR adds the files REF_GTP_LITHO_CLASS.xlsx and REF_GTP_STR_UNIT.xlsx, which are copied from the GeoTOP-zip-file (version 1.6) downloaded from DINOloket. This PR adds a method (_save_excel_files_as_csv, which is never to be called directly by the user) to transform these xlsx-files into csv-files, which are better tracked using version-control, so we can monitor changes in future geotop-versions. REF_GTP_STR_UNIT.csv contains all the lastest stratigraphic units, of which we misinterpreted some (1095) in the list we used to use (geo_eenheden.csv).

One major benefit of this approach is that REF_GTP_STR_UNIT.csv contains a column SEQ_NR, which gives information about the sequence in which layers occur. We can use this SEQ_NR when we transform the geotop data into a layer model, using nlmod.read.geotop.to_model_layers().

@rubencalje rubencalje marked this pull request as ready for review July 10, 2024 06:54
@rubencalje rubencalje merged commit 30b48ff into dev Aug 27, 2024
4 of 5 checks passed
@rubencalje rubencalje deleted the improve_geotop_strat_units branch August 27, 2024 09:45
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