Skip to content

Add regexes to detect common wild foods Livelihood Activities - see HEA-572#177

Merged
rhunwicks merged 1 commit into
mainfrom
HEA-572/regexes_for_wild_foods
Oct 14, 2025
Merged

Add regexes to detect common wild foods Livelihood Activities - see HEA-572#177
rhunwicks merged 1 commit into
mainfrom
HEA-572/regexes_for_wild_foods

Conversation

@rhunwicks

Copy link
Copy Markdown
Contributor

No description provided.

@rhunwicks rhunwicks self-assigned this Oct 14, 2025

Copilot AI left a comment

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.

Pull Request Overview

This PR adds regular expression patterns to detect and categorize wild foods livelihood activities. The changes enable the system to recognize various formats of wild food gathering entries in livelihood activity data.

  • Updates regex patterns to handle wild food gathering activities with specific formatting variations
  • Expands character set support in product patterns to include numbers (1-9)
  • Adds comprehensive test cases for the new wild food detection patterns

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
pipelines/assets/livelihood_activity.py Extends product pattern regex to include numbers 1-9 for better wild food product matching
pipelines/assets/livelihood_activity_regexes.json Adds three new regex patterns to detect wild foods and fish gathering activities
pipelines_tests/test_assets/test_livelihood_activity_regexes.json Adds test cases for avocado, mangoes, okra, tilapia, and fish type 2 gathering activities

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@rhunwicks rhunwicks merged commit 82c6890 into main Oct 14, 2025
13 checks passed
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.

3 participants