Skip to content

Commit 7358b60

Browse files
committed
Address PR feedback see HEA-819
1 parent 12e1a63 commit 7358b60

3 files changed

Lines changed: 11 additions & 8 deletions

File tree

pipelines/assets/livelihood_activity.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ def get_livelihood_activity_regexes() -> list:
240240
"season_pattern": r"(?P<season>season [12]|saison [12]|[12][a-z] season||[12][a-zà-ÿ] saison|r[eé]colte principale|principale r[eé]colte|gu|deyr+?)", # NOQA: E501
241241
"additional_identifier_pattern": r"\(?(?P<additional_identifier>rainfed|irrigated|pluviale?|irriguée|submersion libre|submersion contrôlée|flottant)\)?",
242242
"unit_of_measure_pattern": r"(?P<unit_of_measure_id>[a-z]+)",
243-
"nbr_pattern": r"(?:n[b|o]r?)\.?",
243+
"nbr_pattern": r"(?:n[bo]?r?e?|no)\.?",
244244
"vendu_pattern": r"(?:quantité )?vendu(?:e|s|ss|es|ses)?",
245245
"separator_pattern": r" ?[:-]?",
246246
}

pipelines/assets/livelihood_activity_regexes.json

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -872,13 +872,7 @@
872872
"unit_multiple"
873873
],
874874
[
875-
"(?:petit commerce|petty trade): (?:nbr?\\.? (?:pers|de personnes|personnes|people)(?: ?/ ?| par | per )(?:ménage|mènage|hh)|nbre de personnes par (?:ménage|mènage)|no\\.? people per hh)",
876-
null,
877-
true,
878-
"people_per_household"
879-
],
880-
[
881-
"{product_pattern}{separator_pattern} {nbr_pattern} (?:people per hh|personnes par ménage|personnes par mènage|pers par mènage)",
875+
"{product_pattern}{separator_pattern} {nbr_pattern} (?:de )?(?:pers|personnes|people)(?: ?/ ?| par | per )(?:ménage|mènage|hh)",
882876
null,
883877
true,
884878
"people_per_household"

pipelines_tests/test_assets/test_livelihood_activity_regexes.json

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -868,38 +868,47 @@
868868
},
869869
"petit commerce: nbre de personnes par mènage": {
870870
"is_start": true,
871+
"product_id": "petit commerce",
871872
"attribute": "people_per_household"
872873
},
873874
"petit commerce: nbr. pers par mènage": {
874875
"is_start": true,
876+
"product_id": "petit commerce",
875877
"attribute": "people_per_household"
876878
},
877879
"petit commerce: nb de personnes par ménage": {
878880
"is_start": true,
881+
"product_id": "petit commerce",
879882
"attribute": "people_per_household"
880883
},
881884
"petit commerce: nb de personnes/ménage": {
882885
"is_start": true,
886+
"product_id": "petit commerce",
883887
"attribute": "people_per_household"
884888
},
885889
"petit commerce: nbr de personnes par ménage": {
886890
"is_start": true,
891+
"product_id": "petit commerce",
887892
"attribute": "people_per_household"
888893
},
889894
"petit commerce: nbr. personnes par ménage": {
890895
"is_start": true,
896+
"product_id": "petit commerce",
891897
"attribute": "people_per_household"
892898
},
893899
"petit commerce: nb. personnes par ménage": {
894900
"is_start": true,
901+
"product_id": "petit commerce",
895902
"attribute": "people_per_household"
896903
},
897904
"petty trade: no. people per hh": {
898905
"is_start": true,
906+
"product_id": "petty trade",
899907
"attribute": "people_per_household"
900908
},
901909
"petty trade: nbr. people per hh": {
902910
"is_start": true,
911+
"product_id": "petty trade",
903912
"attribute": "people_per_household"
904913
}
905914
}

0 commit comments

Comments
 (0)