Skip to content

Generating derived data sources from habitatmap_2025#84

Open
florisvdh wants to merge 34 commits intomainfrom
habmap_2025_derivs
Open

Generating derived data sources from habitatmap_2025#84
florisvdh wants to merge 34 commits intomainfrom
habmap_2025_derivs

Conversation

@florisvdh
Copy link
Copy Markdown
Member

@florisvdh florisvdh commented Mar 25, 2026

This PR is work in progress.

For now, @ToonHub and @cecileherr it's useful already to have a look at the habitatmap_stdized code that derives the new columns year_assessment and method_assessment in order to fix #74 (these column names can also be reconsidered). I'm also waiting for some feedback on "mk" and "n" codes to further amend this. (UPDATE 26 March: is tackled in e8de9a9 and 8be0675) The code has been derived from initial code created by @WardTamsyn.

Also fixes #72.

This PR will grow in steps and I may call for some feedback intermittently.

In this top comment, the most up to date rendered HTML files are attached:

WardTamsyn and others added 24 commits March 25, 2026 09:58
An essential aspect is that these attributes better go to the
object with the geometries, not to the long table with the types.
@florisvdh florisvdh requested review from ToonHub and cecileherr March 26, 2026 16:44
@florisvdh
Copy link
Copy Markdown
Member Author

florisvdh commented Mar 26, 2026

@cecileherr @ToonHub a first release candidate of habitatmap_stdized_2025_v1 has been finished in 37e58c1 and is ready for review. The corresponding HTML is attached in the top comment of this PR.

Notable updates for the 2025 version are:

  • add two more columns to the polygon layer: year_assessment and method_assessment
  • remove some package dependencies
  • refactor document sections
  • apply tidyverse code style with {styler}

The data source version is supported by {n2khab} in branch update_for_habmap_2025 (see inbo/n2khab#215).

Comment thread src/generate_habitatmap_stdized/10_generate_habmap_stdized.Rmd Outdated
Comment thread src/generate_habitatmap_stdized/10_generate_habmap_stdized.Rmd
Copy link
Copy Markdown
Contributor

@ToonHub ToonHub left a comment

Choose a reason for hiding this comment

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

Good job!

@cecileherr cecileherr marked this pull request as ready for review March 30, 2026 07:13
Copy link
Copy Markdown
Collaborator

@cecileherr cecileherr left a comment

Choose a reason for hiding this comment

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

I did not check activate.R, but the rest of the code looks fine to me.
Many thanks for the update!

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.

Add year column in future processed versions of habitatmap ? Next version habitatmap_stdized (2025?): remove redundant geometry checks/fixes

4 participants