Skip to content

DE-BB: leave out the NBF records, which had no crop code - #303

Open
saidy-moregeo wants to merge 2 commits into
fiboa:mainfrom
saidy-moregeo:de_bb-nbf
Open

saidy-moregeo wants to merge 2 commits into
fiboa:mainfrom
saidy-moregeo:de_bb-nbf

Conversation

@saidy-moregeo

@saidy-moregeo saidy-moregeo commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Resolves issue #21

# but it should be reconsidered in the future.
column_migrations = {"code": lambda col: col.fillna("").astype(str)}
# NBF (nicht-beihilfefähige Fläche) are ineligible patches, not fields, and carry no crop code
column_filters = {"tf_typ": lambda col: col != "NBF"}

@m-mohr m-mohr Sep 18, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you verify what NBF (nicht-beihilfefähige Fläche) means in Brandenburg?
NBF can be (small) fields generally which just don't get subsidies, but can also be various other things like buildings etc.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes. Brandenburg's DFBK description defines NBF as areas inside a field block that are "nicht landwirtschaftlich nutzbar und kein förderfähiges Landschaftselement". Fields that are temporarily not eligible are a separate type (NAF) and keep their crop code.

@ivorbosloper ivorbosloper left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me

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.

de_bb: 56,640 fields carry an empty crop code, which the non-null rule does not see

3 participants