Skip to content

Preserve culture_de = "allg." when resolving leaf cultures#15

Open
MarcelMagro wants to merge 1 commit into
agroscope-ch:mainfrom
MarcelMagro:marcel_resolve_cultures
Open

Preserve culture_de = "allg." when resolving leaf cultures#15
MarcelMagro wants to merge 1 commit into
agroscope-ch:mainfrom
MarcelMagro:marcel_resolve_cultures

Conversation

@MarcelMagro

Copy link
Copy Markdown
Contributor

When using resolve_culture_allg = TRUE in resolve_cultures(), records with culture_de = "allg." are expanded to the corresponding leaf cultures based on application_area_de.

Previously, the value of culture_de was replaced by the resolved leaf culture. This has been changed. The original value "allg." is now kept by temporarily storing it and restoring it after the leaf cultures have been resolved.

See Example 5.

…ds with culture_de = "allg." are expanded to the corresponding leaf_culture_de based on the application_area_de. Previously, the culture_de column was also overwritten with the resolved leaf_culture_de, replacing the original value "allg.".

This is now changed. A temporary helper column now stores whether the original value of culture_de was "allg.". After resolving the leaf cultures, the original value "allg." is restored in the culture_de column for these records. see example 5.
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.

1 participant