hestia-earth-models 0.65.10__py3-none-any.whl → 0.65.11__py3-none-any.whl

hestia_earth/models/hestia/landCover.py

@@ -82,7 +82,8 @@ LOOKUPS = {
         "Permanent meadows and pastures"
     ],
     "crop": ["cropGroupingFaostatArea", "IPCC_LAND_USE_CATEGORY"],
-    "landCover": ["cropGroupingFaostatProduction", "FAOSTAT_LAND_AREA_CATEGORY"]
+    "landCover": ["cropGroupingFaostatProduction", "FAOSTAT_LAND_AREA_CATEGORY"],
+    "property": "CALCULATE_TOTAL_LAND_COVER_SHARE_SEPARATELY"
 }
 MODEL_KEY = 'landCover'
 
@@ -120,6 +121,13 @@ def site_area_sum_to_100(dict_of_percentages: dict):
          math.isclose(sum(dict_of_percentages.values()), 0.0, rel_tol=0.01))
 
 
+def _should_group_landCover(term: dict):
+    return any(
+        bool(get_lookup_value(lookup_term=prop.get("term", {}), column="CALCULATE_TOTAL_LAND_COVER_SHARE_SEPARATELY"))
+        for prop in term.get("properties", [])
+    )
+
+
 def get_changes(country_id: str, end_year: int) -> dict:
     """
     For each entry in ALL_LAND_USE_TERMS, creates a key: value in output dictionary, also TOTAL
@@ -714,8 +722,7 @@ def _no_prior_land_cover_data(nodes: list, end_date: str) -> bool:
     return len(previous_nodes) == 0
 
 
-def _should_run(site: dict) -> tuple[bool, dict]:
-    management_nodes = filter_list_term_type(site.get("management", []), TermTermType.LANDCOVER)
+def _should_run(site: dict, management_nodes: list) -> tuple[bool, dict]:
     summarised_nodes = _collect_land_use_types(management_nodes)
     allowed_land_use_types = [ANNUAL_CROPLAND, PERMANENT_CROPLAND, PERMANENT_PASTURE]
     relevant_nodes = sorted(
@@ -751,8 +758,11 @@ def _should_run(site: dict) -> tuple[bool, dict]:
 
 
 def run(site: dict) -> list:
-    should_run, site_area = _should_run(site)
-    management_nodes = filter_list_term_type(site.get("management", []), TermTermType.LANDCOVER)
+    management_nodes = [
+        node for node in filter_list_term_type(site.get("management", []), TermTermType.LANDCOVER)
+        if not _should_group_landCover(node)
+    ]
+    should_run, site_area = _should_run(site=site, management_nodes=management_nodes)
     return _run_make_management_nodes(
         existing_nodes=management_nodes,
         percentage_transformed_from=site_area,

hestia_earth/models/hestia/resourceUse_utils.py

@@ -8,7 +8,7 @@ from dateutil.relativedelta import relativedelta
 from hestia_earth.schema import TermTermType
 from hestia_earth.utils.tools import to_precision
 
-from hestia_earth.models.log import logRequirements, logShouldRun
+from hestia_earth.models.log import logRequirements, logShouldRun, debugValues
 from hestia_earth.models.utils.blank_node import _gapfill_datestr, DatestrGapfillMode, DatestrFormat, _str_dates_match
 from hestia_earth.models.utils.impact_assessment import get_site
 from hestia_earth.models.utils.indicator import _new_indicator
@@ -78,11 +78,15 @@ def should_run(
         node for node in impact_assessment.get("emissionsResourceUse", [])
         if node.get("term", {}).get("@id", "") == _RESOURCE_USE_TERM_ID and node.get("value", -1) >= 0
     ]
-
     filtered_management_nodes = [
         node for node in site.get("management", [])
         if node.get("value", -1) >= 0 and node.get("term", {}).get("termType", "") == TermTermType.LANDCOVER.value
     ]
+    land_occupation_during_cycle_found = any(
+        node.get("term", {}).get("@id") in
+        {node.get("landCover", {}).get("@id") for node in relevant_emission_resource_use}
+        for node in filtered_management_nodes
+    )
     current_node_index = next(
         (i for i, node in enumerate(filtered_management_nodes)
          if _str_dates_match(node.get("endDate", ""), impact_assessment.get("endDate", ""))),
@@ -96,19 +100,18 @@ def should_run(
         historic_date_offset=historic_date_offset
     )
 
-    logRequirements(
-        log_node=impact_assessment,
-        model=MODEL,
-        term_id=term_id,
-        site=site
-    )
+    logRequirements(impact_assessment, model=MODEL, term=term_id,
+                    closest_date=closest_date_str,
+                    land_occupation_during_cycle_found=land_occupation_during_cycle_found,
+                    land_cover_term_id=(current_node or {}).get('term', {}).get('@id'))
 
     should_run_result = all([
-        relevant_emission_resource_use != [],
+        relevant_emission_resource_use,
+        land_occupation_during_cycle_found,
         current_node,
         close_date_found
     ])
-    logShouldRun(site, MODEL, term=term_id, should_run=should_run_result)
+    logShouldRun(impact_assessment, MODEL, term=term_id, should_run=should_run_result)
 
     return should_run_result, current_node, closest_date_str
 
@@ -126,6 +129,7 @@ def _get_land_occupation_for_land_use_type(impact_assessment: dict, ipcc_land_us
 
 def _calculate_indicator_value(
     impact_assessment: dict,
+    term_id: str,
     management_nodes: list,
     ipcc_land_use_category: str,
     previous_land_cover_id: str,
@@ -143,6 +147,13 @@ def _calculate_indicator_value(
         node.get("value", 0) for node in management_nodes
         if node.get("term", {}).get("@id", "") == previous_land_cover_id
     )
+
+    debugValues(impact_assessment, model=MODEL, term=term_id,
+                ipcc_land_use_category=ipcc_land_use_category,
+                land_occupation_for_cycle=land_occupation_for_cycle,
+                historical_land_use=historical_land_use,
+                historic_date_offset=historic_date_offset)
+
     return ((land_occupation_for_cycle * historical_land_use) / 100) / historic_date_offset
 
 
@@ -164,6 +175,7 @@ def _run_calculate_transformation(
             previous_land_cover_id=previous_land_cover_id,
             value=_calculate_indicator_value(
                 impact_assessment=impact_assessment,
+                term_id=term_id,
                 management_nodes=[
                     node for node in site.get("management", [])
                     if _str_dates_match(node.get("endDate", ""), closest_date_str)
@@ -191,10 +203,10 @@ def run_resource_use(
         historic_date_offset=historic_date_offset
     )
     return _run_calculate_transformation(
+        impact_assessment=impact_assessment,
+        site=site,
         term_id=term_id,
         current_node=current_node,
         closest_date_str=closest_date_str,
-        site=site,
-        impact_assessment=impact_assessment,
         historic_date_offset=historic_date_offset
     ) if _should_run else []

hestia_earth/models/hestia/utils.py

@@ -43,6 +43,5 @@ def crop_ipcc_land_use_category(
     return get_lookup_value(
         lookup_term={"@id": crop_term_id, "type": "Term", "termType": lookup_term_type},
         column='IPCC_LAND_USE_CATEGORY',
-        model=MODEL,
-        term={"@id": crop_term_id, "type": "Term", "termType": lookup_term_type}
+        model=MODEL
     )

hestia_earth/models/ipcc2019/animal/pastureGrass.py

@@ -233,7 +233,7 @@ def _run_practice(animal: dict, values: dict, meanDE: float, meanECHHV: float, R
             'weightAtOneYear',
             'weightAtSlaughter'
         ])
-        has_positive_feed_values = all([NEm_feed > 0, NEg_feed > 0])
+        has_positive_feed_values = all([NEm_feed >= 0, NEg_feed >= 0])
 
         logRequirements(animal, model=MODEL, term=input_term_id, model_key=MODEL_KEY,
                         feed_logs=log_as_table(log_feed),

hestia_earth/models/ipcc2019/pastureGrass.py

@@ -212,7 +212,7 @@ def _run_practice(cycle: dict, meanDE: float, meanECHHV: float, REM: float, REG:
             'weightAtOneYear',
             'weightAtSlaughter'
         ])
-        has_positive_feed_values = all([NEm_feed > 0, NEg_feed > 0])
+        has_positive_feed_values = all([NEm_feed >= 0, NEg_feed >= 0])
 
         logRequirements(cycle, model=MODEL, term=input_term_id, model_key=MODEL_KEY,
                         feed_logs=log_as_table(log_feed),

hestia_earth/models/mocking/search-results.json

@@ -2022,13 +2022,13 @@
         "@type": "Term",
         "name": "Other natural vegetation",
         "@id": "otherNaturalVegetation",
-        "_score": 43.05355
+        "_score": 43.053535
       },
       {
         "@type": "Term",
         "name": "Agri-food processor",
         "@id": "agriFoodProcessor",
-        "_score": 42.06176
+        "_score": 42.061752
       },
       {
         "@type": "Term",
@@ -2046,7 +2046,7 @@
         "@type": "Term",
         "name": "Permanent pasture",
         "@id": "permanentPasture",
-        "_score": 28.69685
+        "_score": 28.696854
       },
       {
         "@type": "Term",
@@ -2070,7 +2070,7 @@
         "@type": "Term",
         "name": "Forest",
         "@id": "forest",
-        "_score": 20.619314
+        "_score": 20.619322
       },
       {
         "@type": "Term",
@@ -2082,7 +2082,7 @@
         "@type": "Term",
         "name": "Other land",
         "@id": "otherLand",
-        "_score": 19.758362
+        "_score": 19.758368
       },
       {
         "@type": "Term",

hestia_earth/models/pooreNemecek2018/freshwaterWithdrawalsDuringCycle.py

@@ -18,7 +18,6 @@ REQUIREMENTS = {
             "@type": "Cycle",
             "products": [{
                 "@type": "Product",
-                "primary": "True",
                 "value": "> 0",
                 "economicValueShare": "> 0"
             }],

hestia_earth/models/pooreNemecek2018/landOccupationDuringCycle.py

@@ -2,7 +2,8 @@ from hestia_earth.models.log import logRequirements, logShouldRun
 from hestia_earth.models.utils.indicator import _new_indicator
 from hestia_earth.models.utils.impact_assessment import get_product, get_site
 from hestia_earth.models.utils.cycle import land_occupation_per_kg
-from hestia_earth.models.utils.site import get_land_cover_term_id
+from hestia_earth.models.utils.site import get_land_cover_term_id as get_landCover_term_id_from_site_type
+from hestia_earth.models.utils.crop import get_landCover_term_id
 from . import MODEL
 
 REQUIREMENTS = {
@@ -16,7 +17,6 @@ REQUIREMENTS = {
                     "cycleDuration": "",
                     "products": [{
                         "@type": "Product",
-                        "primary": "True",
                         "value": "> 0",
                         "economicValueShare": "> 0"
                     }],
@@ -46,8 +46,8 @@ RETURNS = {
 TERM_ID = 'landOccupationDuringCycle'
 
 
-def _indicator(term_id: str, value: float, land_covert_term_id: str):
-    indicator = _new_indicator(term_id, MODEL, land_covert_term_id)
+def _indicator(term_id: str, value: float, land_cover_term_id: str):
+    indicator = _new_indicator(term_id, MODEL, land_cover_term_id)
     indicator['value'] = value
     return indicator
 
@@ -56,18 +56,21 @@ def _should_run(impact_assessment: dict):
     product = get_product(impact_assessment)
     cycle = impact_assessment.get('cycle', {})
     site = get_site(impact_assessment)
-    land_covert_term_id = get_land_cover_term_id(site.get('siteType'))
+    land_cover_from_product = get_landCover_term_id(product.get('term', {}))
+    land_cover_from_site = get_landCover_term_id_from_site_type(site.get('siteType'))
+    land_cover_term_id = land_cover_from_product or land_cover_from_site
     land_occupation_m2_kg = land_occupation_per_kg(MODEL, TERM_ID, cycle, site, product)
 
     logRequirements(impact_assessment, model=MODEL, term=TERM_ID,
                     land_occupation_kg=land_occupation_m2_kg,
-                    land_covert_term_id=land_covert_term_id)
+                    land_cover_from_product=land_cover_from_product,
+                    land_cover_from_site=land_cover_from_site)
 
-    should_run = all([land_covert_term_id, land_occupation_m2_kg is not None])
+    should_run = all([land_cover_term_id, land_occupation_m2_kg is not None])
     logShouldRun(impact_assessment, MODEL, TERM_ID, should_run)
-    return should_run, land_occupation_m2_kg, land_covert_term_id
+    return should_run, land_occupation_m2_kg, land_cover_term_id
 
 
 def run(impact_assessment: dict):
-    should_run, land_occupation_kg, land_covert_term_id = _should_run(impact_assessment)
-    return [_indicator(TERM_ID, land_occupation_kg, land_covert_term_id)] if should_run else []
+    should_run, land_occupation_kg, land_cover_term_id = _should_run(impact_assessment)
+    return [_indicator(TERM_ID, land_occupation_kg, land_cover_term_id)] if should_run else []

hestia_earth/models/site/management.py

@@ -31,11 +31,11 @@ from hestia_earth.models.utils import _include, _omit, group_by
 from hestia_earth.models.utils.management import _new_management
 from hestia_earth.models.utils.term import get_lookup_value
 from hestia_earth.models.utils.blank_node import condense_nodes, DatestrFormat, _gapfill_datestr, DatestrGapfillMode
+from hestia_earth.models.utils.crop import get_landCover_term_id
 from hestia_earth.models.utils.site import (
     related_cycles, get_land_cover_term_id as get_landCover_term_id_from_site_type
 )
 from . import MODEL
-from ..utils.crop import get_landCover_term_id
 
 REQUIREMENTS = {
     "Site": {

hestia_earth/models/utils/blank_node.py

@@ -73,7 +73,7 @@ def lookups_logs(model: str, blank_nodes: list, lookups_per_termType: dict, **lo
 
         def _reduce_lookups_logs(logs: dict, column: str):
             lookup_value = get_lookup_value(term, column, model=model, **log_args)
-            return logs | {column: lookup_value}
+            return logs | {column: str(lookup_value).replace(':', '-')}
 
         return reduce(_reduce_lookups_logs, lookups, {'id': term_id})
 

hestia_earth/models/utils/crop.py

@@ -67,6 +67,6 @@ def get_landCover_term_id(lookup_term: dict, **log_args) -> str:
     value = get_lookup_value(lookup_term, 'landCoverTermId', **log_args)
     return (
         lookup_term.get("@id") if lookup_term.get("termType") == TermTermType.LANDCOVER.value else
-        value.split(';')[0] if value else
+        value.split(';')[0] if isinstance(value, str) else
         None
     )

hestia_earth/models/utils/cycle.py

@@ -291,7 +291,7 @@ def _land_occupation_per_kg(model: str, term_id: str, cycle: dict, product: dict
     return value
 
 
-def land_occupation_per_kg(model: str, term_id: str, cycle: dict, site: dict, primary_product: dict):
+def land_occupation_per_kg(model: str, term_id: str, cycle: dict, site: dict, product: dict):
     """
     Get the land occupation of the cycle per kg in meter square.
 
@@ -305,7 +305,7 @@ def land_occupation_per_kg(model: str, term_id: str, cycle: dict, site: dict, pr
         The `Cycle` as defined in the HESTIA Schema.
     site : dict
         The `Site` as defined in the HESTIA Schema.
-    primary_product : dict
+    product : dict
         The primary `Product` of the `Cycle`.
 
     Returns
@@ -325,7 +325,7 @@ def land_occupation_per_kg(model: str, term_id: str, cycle: dict, site: dict, pr
         SiteSiteType.RIVER_OR_STREAM.value,
         SiteSiteType.SEA_OR_OCEAN.value
     ] else (
-        _land_occupation_per_kg(model, term_id, cycle, primary_product, value) if value is not None else None
+        _land_occupation_per_kg(model, term_id, cycle, product, value) if value is not None else None
     )
 
 

hestia_earth/models/version.py

@@ -1 +1 @@
-VERSION = '0.65.10'
+VERSION = '0.65.11'

{hestia_earth_models-0.65.10.dist-info → hestia_earth_models-0.65.11.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: hestia-earth-models
-Version: 0.65.10
+Version: 0.65.11
 Summary: HESTIA's set of modules for filling gaps in the activity data using external datasets (e.g. populating soil properties with a geospatial dataset using provided coordinates) and internal lookups (e.g. populating machinery use from fuel use). Includes rules for when gaps should be filled versus not (e.g. never gap fill yield, gap fill crop residue if yield provided etc.).
 Home-page: https://gitlab.com/hestia-earth/hestia-engine-models
 Author: HESTIA Team

{hestia_earth_models-0.65.10.dist-info → hestia_earth_models-0.65.11.dist-info}/RECORD

@@ -4,7 +4,7 @@ hestia_earth/models/cache_sites.py,sha256=Llo2SH1Lp-R8x1JRxJ2Ta-vw5RbdUj2FHXUP-c
 hestia_earth/models/log.py,sha256=_zAfyOkL_VknEnMFvcpvenSMghadlDfZhiSx28545Gk,3558
 hestia_earth/models/preload_requests.py,sha256=vK_G1UzhNMhYy7ymnCtHUz_vv3cfApCSKqv29VREEBQ,1943
 hestia_earth/models/requirements.py,sha256=eU4yT443fx7BnaokhrLB_PCizJI7Y6m4auyo8vQauNg,17363
-hestia_earth/models/version.py,sha256=kGCwazFyJPTHO9Kny5rZmqHxPuatjOBEax2liicBscw,20
+hestia_earth/models/version.py,sha256=6LpNoOHn7BXGQSdeOERwYSNo64BVnj0d53V5FW5wWHY,20
 hestia_earth/models/agribalyse2016/__init__.py,sha256=WvK0qCQbnYtg9oZxrACd1wGormZyXibPtpCnIQeDqbw,415
 hestia_earth/models/agribalyse2016/fuelElectricity.py,sha256=rm5ZaRAzJ08m2y4BxkGh-RjudkDWgozmg3XumoRm-fQ,4511
 hestia_earth/models/agribalyse2016/machineryInfrastructureDepreciatedAmountPerCycle.py,sha256=BPjnWmg73i_OxM2ouCdMTWZtPIqyoUAXrvutntyteE0,3390
@@ -206,12 +206,12 @@ hestia_earth/models/haversineFormula/__init__.py,sha256=o155nR-XI67iCSBVNYIu4sPR
 hestia_earth/models/haversineFormula/transport/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 hestia_earth/models/haversineFormula/transport/distance.py,sha256=163KrmKzlEQuKYT1ZvpPgmKlv_-mmvxp0A1_uKya99w,4203
 hestia_earth/models/hestia/__init__.py,sha256=o5vAmPzSaK9XPgL8GCne3-lugfCOgZhHELYolNgqyyY,407
-hestia_earth/models/hestia/landCover.py,sha256=zq9JIJn4qdsX3SvZxmS5qllOl0xinJfrHDR5dsYjDpg,29507
+hestia_earth/models/hestia/landCover.py,sha256=P1kKasnLwmwHyxCMNKxaVEDPdcSBfOuicG-94kLnBro,29846
 hestia_earth/models/hestia/landTransformation100YearAverageDuringCycle.py,sha256=-7ToRvCVPD6AAcjxorPS5jSWio7JAglHrdSS9PPyPqQ,1551
 hestia_earth/models/hestia/landTransformation20YearAverageDuringCycle.py,sha256=TCskVLhYXBMxdeZM-gN4Tdixk5ua7eVn-o5dfIT_H7o,1543
-hestia_earth/models/hestia/resourceUse_utils.py,sha256=1ySn4d-qkDeU8Ss_80l-uOypPoWsmDsqnS6IM8wkI34,7113
+hestia_earth/models/hestia/resourceUse_utils.py,sha256=b-0g_xArf_4Ln7JCqaSdCk7nXonAHYo_mLzmJnGLlhQ,7939
 hestia_earth/models/hestia/seed_emissions.py,sha256=58hmHAgemOuGVIsij-FWF7EsGmEbCSfzFAEfXkXSyEE,11566
-hestia_earth/models/hestia/utils.py,sha256=meymdvm0KZRLYGWeBLSv7QDn6tJHwM8jN8YB6gnwTqU,1537
+hestia_earth/models/hestia/utils.py,sha256=TX53XPLeu3XgBcy9XN8xDD3J7y7fjmVweRNKaEZ-qzE,1455
 hestia_earth/models/impact_assessment/__init__.py,sha256=gTR_PhWps593fPhm-V826VLLrZVH8CNQTqxExB7GGNI,418
 hestia_earth/models/impact_assessment/allocationMethod.py,sha256=Qz41nTtMpDCcPy7PjhVtafE13dfJLX_D3Rg3yNhdY_Q,1279
 hestia_earth/models/impact_assessment/emissions.py,sha256=oHWXFZgvxYktH6gAawEjphyLtPCmH0FWt_mDwphc2I8,4394
@@ -285,7 +285,7 @@ hestia_earth/models/ipcc2019/organicCarbonPerHa.py,sha256=C9ef41fd-gYixEV3sTs9VM
 hestia_earth/models/ipcc2019/organicCarbonPerHa_tier_1_utils.py,sha256=hgMh3p7tflTo06LJjAkoz8btrl8Rc6lYYvZp1ooJIaI,76214
 hestia_earth/models/ipcc2019/organicCarbonPerHa_tier_2_utils.py,sha256=e2O4yZ9bCJWMPQ8s-DEwnaDImGGCnpJya6SO0tWy0RY,63367
 hestia_earth/models/ipcc2019/organicCarbonPerHa_utils.py,sha256=FFcjdkYq5NJwAu4bWZJUao4MoDHvgbXmXCnUNKD-a3w,9842
-hestia_earth/models/ipcc2019/pastureGrass.py,sha256=Hhm9szzsL3otuo-V6fQ_cNyDbG0dlubr4-1L4WefbDE,10304
+hestia_earth/models/ipcc2019/pastureGrass.py,sha256=HWRkpYpGqvMfol2dPb9pFkNzoGj7n6sRpyIjGSVM9lQ,10306
 hestia_earth/models/ipcc2019/pastureGrass_utils.py,sha256=nL31uS3c77PH_5nA2E2MvB1NS2mMvhD341taGl5YWNE,13842
 hestia_earth/models/ipcc2019/utils.py,sha256=MSDMu15D9DnilFUgi4_6jYXC0FaKso3OODauGTMB6hs,6229
 hestia_earth/models/ipcc2019/animal/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -294,7 +294,7 @@ hestia_earth/models/ipcc2019/animal/hoursWorkedPerDay.py,sha256=f-zcslXkbIY9ef-p
 hestia_earth/models/ipcc2019/animal/liveweightGain.py,sha256=UElmAdB4yQEFAOT5RrURn0Yt7U3gB2qiCWvNXyLk8Hw,971
 hestia_earth/models/ipcc2019/animal/liveweightPerHead.py,sha256=cq88VTxPYVPyEr2NLPb0Zmmppn_eMp9Xtw6Nxnta4-M,984
 hestia_earth/models/ipcc2019/animal/milkYieldPerAnimal.py,sha256=XxgSvQLi1Xu6ZpUg1a-7OFZEkItkUK-nVPTDxApUnuY,2777
-hestia_earth/models/ipcc2019/animal/pastureGrass.py,sha256=4vs-KIlCKQhU1EpX0BFlVAQrh1szm154AnrPDs13oE4,12212
+hestia_earth/models/ipcc2019/animal/pastureGrass.py,sha256=M-6SigPr3Yfw-Iq1hVyh38tuh6_J5GR5rfGbjDqEPgU,12214
 hestia_earth/models/ipcc2019/animal/pregnancyRateTotal.py,sha256=MWPnuT4WeKJCGjJ_DAJnYelsX2Ld0C5e8eN__0Ws0E0,982
 hestia_earth/models/ipcc2019/animal/trueProteinContent.py,sha256=9hOg5KAFQ6Q-2_5h8pduYiYg9QBr22UnHdJYmkh8qi0,1023
 hestia_earth/models/ipcc2019/animal/utils.py,sha256=HPOjdJGUxn4Ls4jnr0kD5_w-JrzG1C7_cwdMpCCA1kk,4514
@@ -406,15 +406,15 @@ hestia_earth/models/linkedImpactAssessment/utils.py,sha256=S1zlux02gU2Lajrtoq-zQ
 hestia_earth/models/mocking/__init__.py,sha256=9VX50c-grz-snfd-7MBS0KfF7AadtbKuj7kK6PqtsgE,687
 hestia_earth/models/mocking/build_mock_search.py,sha256=p15ccEUmkmLp1RiGNznxMz3OFHbI8P1-29ExuohiQN8,1355
 hestia_earth/models/mocking/mock_search.py,sha256=ccFe_WrI73JElFmxp4hPNLCX7eeU--lBC1JFR901KJY,1069
-hestia_earth/models/mocking/search-results.json,sha256=_8AfPAX_YcwIYlJpyaHp5sxw4jpNX2Mub58HqO3Szuo,102054
+hestia_earth/models/mocking/search-results.json,sha256=EFCT8ggjUM49bin_Hxq6de9Ceu1BTx0JdQ_LHGVLC8M,102057
 hestia_earth/models/pooreNemecek2018/__init__.py,sha256=nPboL7ULJzL5nJD5q7q9VOZt_fxbKVm8fmn1Az5YkVY,417
 hestia_earth/models/pooreNemecek2018/aboveGroundCropResidueTotal.py,sha256=Qt-mel4dkhK6N5uUOutNOinCTFjbjtGzITaaI0LvYc4,2396
 hestia_earth/models/pooreNemecek2018/belowGroundCropResidue.py,sha256=JT0RybbvWVlo01FO8K0Yj41HrEaJT3Kj1xfayr2X-xw,2315
 hestia_earth/models/pooreNemecek2018/ch4ToAirAquacultureSystems.py,sha256=CxjhFinScTebBGNVheEdbdw36DdHwoPszAnbxtd6_9s,6590
 hestia_earth/models/pooreNemecek2018/excretaKgN.py,sha256=WI0SJJR8uJFfG1hTHDSOeiuTMhAW1fKcm0MX1QEvxcg,6340
 hestia_earth/models/pooreNemecek2018/excretaKgVs.py,sha256=TW0lF752Ljlx7zj1v3TkAYLYASsYsYpEy8KH9y2yTQ8,9446
-hestia_earth/models/pooreNemecek2018/freshwaterWithdrawalsDuringCycle.py,sha256=XP3SAVvvrKCInjkM5GlW8MeW9Q-Ib7vXmJk0PAgR-lc,3982
-hestia_earth/models/pooreNemecek2018/landOccupationDuringCycle.py,sha256=78N513lNtjol0KKddxmG3Pb9vAmledlrWbfURVdLHBA,3139
+hestia_earth/models/pooreNemecek2018/freshwaterWithdrawalsDuringCycle.py,sha256=rBWK_SzR9siorMxNORqB_BGQEadjyvzSh_SKOqRwFV0,3947
+hestia_earth/models/pooreNemecek2018/landOccupationDuringCycle.py,sha256=gSUzPuzzLwQsBiX_Tgx6mWJ4pEB5qhlKoOtaEdmDGQ4,3431
 hestia_earth/models/pooreNemecek2018/longFallowDuration.py,sha256=Wdm6QyOttCFP9Y3OjbaYrvdMmivOmMIT-m5Eg9SM9rY,1511
 hestia_earth/models/pooreNemecek2018/n2OToAirAquacultureSystemsDirect.py,sha256=HJ7IstImGyasIKosK2lQZ-v6Lqt3_aEfZhoiC4CY0rM,2586
 hestia_earth/models/pooreNemecek2018/n2ToAirAquacultureSystems.py,sha256=SoZlogDd7_4kq5S9gc8KmVeIXacWWhaUkWlKTuho_OA,2431
@@ -510,7 +510,7 @@ hestia_earth/models/site/defaultMethodClassification.py,sha256=mqy3P3DLf7cr4-TU_
 hestia_earth/models/site/defaultMethodClassificationDescription.py,sha256=5rm9LUINmHF579HaqWfHZpFTjwZSgglE6TgAwBPs1uc,1250
 hestia_earth/models/site/flowingWater.py,sha256=v3g5722GIA4zQAUQI9yGFiZvFvI1QAVZqlQrY-6_B3A,1731
 hestia_earth/models/site/freshWater.py,sha256=FXs3Vt8V4e-wn325_dwSTOKlZtn5ksNUpvYGDeLJShY,1255
-hestia_earth/models/site/management.py,sha256=38_os7XxhwBrNQKO-1lKhzAg-HR89Fy0jAV5LdC_Ueg,15016
+hestia_earth/models/site/management.py,sha256=GFM8d3AzGUxojLiTpcPnKk0wbYX8RhryNKSZhz8bio4,15034
 hestia_earth/models/site/netPrimaryProduction.py,sha256=UIIQkYd911qVzrWjxBLrC37e-RARIVgDwLdARY9BuLw,1849
 hestia_earth/models/site/organicCarbonPerHa.py,sha256=F2ShinHf0m9qKa1nCYBspsDkRY6jzOl0wM8mSDre22I,14916
 hestia_earth/models/site/organicCarbonPerKgSoil.py,sha256=t--wAshiAKS-JvEKhLFRadGvgSBv5NFZ68jdyms_wh4,1945
@@ -573,15 +573,15 @@ hestia_earth/models/utils/aggregated.py,sha256=01V5RDvO9EZAEiApY7M2dUoR4GcGxvAH5
 hestia_earth/models/utils/animalProduct.py,sha256=M5IunAKGY6oZv3j1Ascl34ywyeLWApqOIlBzbtlA2FE,721
 hestia_earth/models/utils/aquacultureManagement.py,sha256=dxrbC1Xf140cohxTbSw6TxLAnAASWTdNZwBBam4yQnw,171
 hestia_earth/models/utils/array_builders.py,sha256=QEkM7e75Y9aq5yEPEk3R8md36IuvCDkpekdGm3uA0uk,19397
-hestia_earth/models/utils/blank_node.py,sha256=-IURt-nrVCJUk2Q51Ar46iEYv4Cn3aSdlVl1gik5Vvw,51992
+hestia_earth/models/utils/blank_node.py,sha256=m1-ZOHt0ImOzCRZDhr5uI44SrKAmRRXyT_tfUGg5uqk,52015
 hestia_earth/models/utils/cache_sources.py,sha256=MBkrPpjwNiC4ApDjeYVHZjWBbpvAerXRDrMHpjasAZ0,377
 hestia_earth/models/utils/completeness.py,sha256=2-GusD9UycobDZq8y5jar0ZcOjyqnSbzPRT_5XMc4YA,1259
 hestia_earth/models/utils/constant.py,sha256=6wLx8xb2R8HtpEpVy5e-PbioOo7QCu2n-W72fs6OvgE,3411
-hestia_earth/models/utils/crop.py,sha256=szbmQFIfKAgH18PXkwx4FcmSsYOzDM3AHN5z0aYK9OU,2696
+hestia_earth/models/utils/crop.py,sha256=JF9vsXOlViiwsIkV-mQ9iRl4dZGgeP2mrgdAmKuhYaY,2713
 hestia_earth/models/utils/cropResidue.py,sha256=_0Q35CrliJeo31xGHsPWe8A2oHxijdIsOrf3gBEqhlA,612
 hestia_earth/models/utils/cropResidueManagement.py,sha256=nIDFjf39rDD10UHSVudfDyu-EiL261g8jyrgS-2aDKw,347
 hestia_earth/models/utils/currency.py,sha256=f_ArJANb--pZq4LL49SXQ1AMX_oKroqwBXKRRQqZwsM,578
-hestia_earth/models/utils/cycle.py,sha256=uZgqTy4IaCqcSEEHWOcZ0bFBjwGl7gBoedk6HJ5Nryw,16233
+hestia_earth/models/utils/cycle.py,sha256=PmloVCSU4TtXnEe0ltP--bTIZBXVB-rEFJrV6IYsdeo,16209
 hestia_earth/models/utils/descriptive_stats.py,sha256=EMVwFvg2OnZgKRAfireAoWY2EbrSvqR0V0bK9B53p28,1583
 hestia_earth/models/utils/ecoClimateZone.py,sha256=kD5DGActHAfMCJykKQGkwEEicWt7PQlEIX9_PkqXfP0,4265
 hestia_earth/models/utils/emission.py,sha256=H_apu-Og9SJTLVU2lU56IsvSU22-9J7OrqXk1b2qnSE,3638
@@ -818,9 +818,9 @@ tests/models/haversineFormula/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5N
 tests/models/haversineFormula/transport/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/models/haversineFormula/transport/test_distance.py,sha256=hqzIOA1nGao8uiBE16J0ou52McwV4w30ZLpEAqtfi9k,970
 tests/models/hestia/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-tests/models/hestia/test_landCover.py,sha256=TCgZMAd11TjXs0RZHHJzjXSCY_UicFgYPoGjjryScm4,6105
+tests/models/hestia/test_landCover.py,sha256=mcjzAaQy8a2V45lExLuUQiQLpBShTKIn4FK2-lIvy4o,6129
 tests/models/hestia/test_landTransformation100YearAverageDuringCycle.py,sha256=3qa4rWUFqP1VM5-vm_182rhiBYJDxPqJwWtBqJ5K028,956
-tests/models/hestia/test_landTransformation20YearAverageDuringCycle.py,sha256=257nCGseM8IEc7i3c2lvx0AsJOpk5Cy633PlZZQYRGo,956
+tests/models/hestia/test_landTransformation20YearAverageDuringCycle.py,sha256=gMK0kGIucX9nbYPNbvYreCCJ2BmHGjrhx52hcy1uPGc,979
 tests/models/hestia/test_seed_emissions.py,sha256=dCUuJBkhwNFBhhcypQN7eMqrWZ9iGCnypoidO5DfQYw,921
 tests/models/impact_assessment/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/models/impact_assessment/test_allocationMethod.py,sha256=HkUt6oWdEyFSdxMpx8y_Oxk0nO2Fi9B_kUZcVte9tzg,1223
@@ -1017,7 +1017,7 @@ tests/models/pooreNemecek2018/test_ch4ToAirAquacultureSystems.py,sha256=rUxD57yl
 tests/models/pooreNemecek2018/test_excretaKgN.py,sha256=4YSri6_P5YXLx8cCAV4SJD36MEL_wWB8ZK-_AwGn0zQ,4064
 tests/models/pooreNemecek2018/test_excretaKgVs.py,sha256=h41KnwQDo4Jp3HrbXTGK-qLU4ZPsWzhDZ5jIwDrmLuo,3256
 tests/models/pooreNemecek2018/test_freshwaterWithdrawalsDuringCycle.py,sha256=5QxDU6VzcwWup3Nuhn32kJoIlAkqcSggGr4CmoRqrFA,1623
-tests/models/pooreNemecek2018/test_landOccupationDuringCycle.py,sha256=MrgxJIuojKCuwCKX6d_KMsg8dTkfdxWwZPnCtjFZDwo,1873
+tests/models/pooreNemecek2018/test_landOccupationDuringCycle.py,sha256=rEhRHN6h6i4l89mZpyHE-pfsFM8lQUHw__jLu6FKz54,1728
 tests/models/pooreNemecek2018/test_longFallowDuration.py,sha256=kelZajIbKyvVm1vX_grRZy0IUrtejGI5GPn03qbElnw,925
 tests/models/pooreNemecek2018/test_n2OToAirAquacultureSystemsDirect.py,sha256=4YAoUhIwfEmRe2B5BvNg247VeB7UXHNeZqr2mRtJA7Y,2207
 tests/models/pooreNemecek2018/test_n2ToAirAquacultureSystems.py,sha256=YqwAvbYfLVamfkUJUwGTEWEnz8ts66jVF8831QcWNkg,2200
@@ -1166,7 +1166,7 @@ tests/models/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuF
 tests/models/utils/test_array_builders.py,sha256=uHxO4beJhdJD-1lI5aPEAsYziQP8SN7AJ9PEY67DpaI,8417
 tests/models/utils/test_blank_node.py,sha256=G_hH-Gk3__1ZtHQcsdYMbUMwVNSTmYcVBs5jcReIYGM,39945
 tests/models/utils/test_cache_sources.py,sha256=xcGMVbYoV23YC4HLBTL_4qZP1ME2cp3DFebgJMxI2TE,721
-tests/models/utils/test_crop.py,sha256=ve_BqT2XuLTWAH-8YOnVY4nzMUSXYYXPB8S-5RtGlf4,463
+tests/models/utils/test_crop.py,sha256=d508vQdtB_Q_6hEahNueXed5PaEiLeN7ScbwZIKCiDI,862
 tests/models/utils/test_cropResidueManagement.py,sha256=RQt8lexeJzCyxZceIutgDpw7BpcqmjsUB0C0yZC2QpY,930
 tests/models/utils/test_currency.py,sha256=BPsojPsY9QW2aj8vgbjkPQXU8GU6wDwwtPZ3HdC4KTU,277
 tests/models/utils/test_dataCompleteness.py,sha256=mW6rA7ddhtjZsLxwo5xDnvBdxmqTsivy4RTIU-2Lvk0,1713
@@ -1208,8 +1208,8 @@ tests/orchestrator/strategies/run/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeR
 tests/orchestrator/strategies/run/test_add_blank_node_if_missing.py,sha256=lGqeebvgAwGathB8NLZ14Js5JV_-KyHueaD6I8IH8mU,3615
 tests/orchestrator/strategies/run/test_add_key_if_missing.py,sha256=hKwvk1ohcBVnQUCTiDhRW99J0xEa29BpwFi1KC0yWLE,329
 tests/orchestrator/strategies/run/test_always.py,sha256=w5-Dhp6yLzgZGAeMRz3OrqZbbAed9gZ1O266a3z9k7w,134
-hestia_earth_models-0.65.10.dist-info/LICENSE,sha256=TD25LoiRJsA5CPUNrcyt1PXlGcbUGFMAeZoBcfCrCNE,1154
-hestia_earth_models-0.65.10.dist-info/METADATA,sha256=1B8T70nCeba3dfHkL6iNd8l0VARIGLSxUYTfw5LFcXA,4047
-hestia_earth_models-0.65.10.dist-info/WHEEL,sha256=tZoeGjtWxWRfdplE7E3d45VPlLNQnvbKiYnx7gwAy8A,92
-hestia_earth_models-0.65.10.dist-info/top_level.txt,sha256=1dqA9TqpOLTEgpqa-YBsmbCmmNU1y56AtfFGEceZ2A0,19
-hestia_earth_models-0.65.10.dist-info/RECORD,,
+hestia_earth_models-0.65.11.dist-info/LICENSE,sha256=TD25LoiRJsA5CPUNrcyt1PXlGcbUGFMAeZoBcfCrCNE,1154
+hestia_earth_models-0.65.11.dist-info/METADATA,sha256=36kRKh8wpenfujTV4awKBVoP9is-KFfwyA8e6IRE6YU,4047
+hestia_earth_models-0.65.11.dist-info/WHEEL,sha256=tZoeGjtWxWRfdplE7E3d45VPlLNQnvbKiYnx7gwAy8A,92
+hestia_earth_models-0.65.11.dist-info/top_level.txt,sha256=1dqA9TqpOLTEgpqa-YBsmbCmmNU1y56AtfFGEceZ2A0,19
+hestia_earth_models-0.65.11.dist-info/RECORD,,

tests/models/hestia/test_landCover.py

@@ -197,7 +197,8 @@ def test_get_sums_of_crop_expansion():
         "gbr_example",
         "malaysia",
         "prior_landCover",
-        "nigeria_cassava"
+        "nigeria_cassava",
+        "cover_crops",
     ]
 )
 @patch(f"{CLASS_PATH}._new_management", side_effect=fake_new_management)

tests/models/hestia/test_landTransformation20YearAverageDuringCycle.py

@@ -14,7 +14,8 @@ fixtures_folder = f"{fixtures_path}/{MODEL}/{TERM_ID}"
 @pytest.mark.parametrize(
     "subfolder",
     [
-        "maize_brazil"
+        "maize_brazil",
+        "orange_iran"
     ]
 )
 @patch("hestia_earth.models.hestia.resourceUse_utils._new_indicator", side_effect=fake_new_indicator)

tests/models/pooreNemecek2018/test_landOccupationDuringCycle.py

@@ -9,7 +9,7 @@ class_path = f"hestia_earth.models.{MODEL}.{TERM_ID}"
 fixtures_folder = f"{fixtures_path}/{MODEL}/{TERM_ID}"
 
 
-@patch(f"{class_path}.get_land_cover_term_id", return_value='cropland')
+@patch(f"{class_path}.get_landCover_term_id", return_value='cropland')
 @patch(f"{class_path}.land_occupation_per_kg", return_value=None)
 def test_should_run(mock_land_occupation, *args):
     # with a cycle and functionalUnit = 1 ha => no run
@@ -23,7 +23,6 @@ def test_should_run(mock_land_occupation, *args):
     assert should_run is True
 
 
-@patch(f"{class_path}.get_land_cover_term_id", return_value='cropland')
 @patch(f"{class_path}._new_indicator", side_effect=fake_new_indicator)
 def test_run(*args):
     with open(f"{fixtures_folder}/impact-assessment.jsonld", encoding='utf-8') as f:
@@ -36,7 +35,6 @@ def test_run(*args):
     assert value == expected
 
 
-@patch(f"{class_path}.get_land_cover_term_id", return_value='cropland')
 @patch(f"{class_path}._new_indicator", side_effect=fake_new_indicator)
 def test_run_with_plantation(*args):
     with open(f"{fixtures_folder}/with-orchard-crop/impact-assessment.jsonld", encoding='utf-8') as f:

tests/models/utils/test_crop.py

@@ -1,6 +1,7 @@
-from hestia_earth.schema import SiteSiteType
+import pytest
+from hestia_earth.schema import SiteSiteType, TermTermType
 
-from hestia_earth.models.utils.crop import valid_site_type
+from hestia_earth.models.utils.crop import valid_site_type, get_landCover_term_id
 
 class_path = 'hestia_earth.models.utils.crop'
 
@@ -13,3 +14,14 @@ def test_valid_site_type():
     cycle['site']['siteType'] = SiteSiteType.PERMANENT_PASTURE.value
     assert not valid_site_type(cycle)
     assert not valid_site_type(site, True) is True
+
+
+@pytest.mark.parametrize(
+    'term,expected',
+    [
+        ({'termType': TermTermType.CROP.value, '@id': 'wheatGrain'}, 'wheatPlant'),
+        ({'termType': TermTermType.SEED.value, '@id': 'saplings'}, None),
+    ]
+)
+def test_get_landCover_term_id(term: dict, expected: str):
+    assert get_landCover_term_id(term) == expected, term.get('@id')