Merge branch 'main' into fix-completeness-text

JanReifenrath · web-flow · commit 2398871311be · 2025-01-13T16:42:34.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,9 +1,17 @@
 # Changelog
 
-## Current Main 
+## Current Main
+
+## Release  1.8.2 
 
 - PR template: Hurl maintenance is mentioned and wording with respect to mergeability is clarified ([#860])
 
+### Bug Fixes
+
+- attribute-completeness: custom filters are now correctly combined with topic filter ([#865])
+
+[#865]: https://github.com/GIScience/ohsome-quality-api/pull/865
+
 [#860]: https://github.com/GIScience/ohsome-quality-api/issues/860
 
 
diff --git a/ohsome_quality_api/__init__.py b/ohsome_quality_api/__init__.py
@@ -1,4 +1,4 @@
-__version__ = "1.8.1"
+__version__ = "1.8.2"
 __title__ = "ohsome quality API"
 __description__ = "Data quality estimations for OpenStreetMap"
 __author__ = "ohsome team"
diff --git a/ohsome_quality_api/attributes/definitions.py b/ohsome_quality_api/attributes/definitions.py
@@ -52,16 +52,12 @@ def get_attribute_preset(topic_key: str) -> List[Attribute]:
         ) from error
 
 
-def build_attribute_filter(attribute_key: List[str], topic_key: str) -> str:
+def build_attribute_filter(attribute_key: List[str] | str, topic_key: str) -> str:
     """Build attribute filter for ohsome API query."""
     attributes = get_attributes()
     try:
         if isinstance(attribute_key, str):
-            return (
-                get_topic_preset(topic_key).filter
-                + " and "
-                + attributes[topic_key][attribute_key].filter
-            )
+            return get_topic_preset(topic_key).filter + " and (" + attribute_key + ")"
         else:
             attribute_filter = get_topic_preset(topic_key).filter
             for key in attribute_key:
diff --git a/ohsome_quality_api/indicators/attribute_completeness/indicator.py b/ohsome_quality_api/indicators/attribute_completeness/indicator.py
@@ -67,6 +67,11 @@ def __init__(
                     for k in self.attribute_keys
                 ]
             )
+        else:
+            self.attribute_filter = build_attribute_filter(
+                self.attribute_filter,
+                self.topic.key,
+            )
 
     async def preprocess(self) -> None:
         # Get attribute filter
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "ohsome_quality_api"
-version = "1.8.1"
+version = "1.8.2"
 description = "Data quality estimations for OpenStreetMap."
 authors = ["ohsome team <ohsome@heigit.org>"]
 homepage = "https://api.quality.ohsome.org"
diff --git a/scripts/release.sh b/scripts/release.sh
@@ -5,8 +5,8 @@
 
 
 # please adjust these version names first
-export OLD_VERSION=1.8.0
-export NEW_VERSION=1.8.1
+export OLD_VERSION=1.8.1
+export NEW_VERSION=1.8.2
 
 
 # exit immediately if a command exits with a non-zero status
diff --git a/tests/integrationtests/api/test_indicators_attribute_completeness.py b/tests/integrationtests/api/test_indicators_attribute_completeness.py
diff --git a/tests/integrationtests/approved/api/test_indicators_atribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers0-schema0].approved.txt b/tests/integrationtests/approved/api/test_indicators_atribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers0-schema0].approved.txt
@@ -1 +1 @@
-Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 9: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
+Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 63: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
diff --git a/tests/integrationtests/approved/api/test_indicators_atribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers1-schema1].approved.txt b/tests/integrationtests/approved/api/test_indicators_atribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers1-schema1].approved.txt
@@ -1 +1 @@
-Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 9: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
+Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 63: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
diff --git a/tests/integrationtests/approved/api/test_indicators_attribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers0-schema0].approved.txt b/tests/integrationtests/approved/api/test_indicators_attribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers0-schema0].approved.txt
@@ -1 +1 @@
-Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 9: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
+Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 63: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
diff --git a/tests/integrationtests/approved/api/test_indicators_attribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers1-schema1].approved.txt b/tests/integrationtests/approved/api/test_indicators_attribute_completeness.py-test_indicators_attribute_completeness_filter_invalid[headers1-schema1].approved.txt
@@ -0,0 +1 @@
+Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 63: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.
diff --git a/tests/integrationtests/approved/indicators/test_attribute_completeness.py-TestCalculation-test_calculate[indicator1].approved.txt b/tests/integrationtests/approved/indicators/test_attribute_completeness.py-TestCalculation-test_calculate[indicator1].approved.txt
@@ -1 +1 @@
-60.9% of all "building count" features (all: 30263 elements) in your area of interest have the selected additional attribute Height (matched: 18444 elements). 
+39.8% of all "building count" features (all: 30263 elements) in your area of interest have the selected additional attribute Height (matched: 12059 elements). 
diff --git a/tests/integrationtests/fixtures/vcr_cassettes/api/test_indicators_atribute_completeness.yaml b/tests/integrationtests/fixtures/vcr_cassettes/api/test_indicators_atribute_completeness.yaml
diff --git a/tests/integrationtests/fixtures/vcr_cassettes/api/test_indicators_attribute_completeness.yaml b/tests/integrationtests/fixtures/vcr_cassettes/api/test_indicators_attribute_completeness.yaml
diff --git a/tests/integrationtests/fixtures/vcr_cassettes/indicators/test_attribute_completeness.yaml b/tests/integrationtests/fixtures/vcr_cassettes/indicators/test_attribute_completeness.yaml
diff --git a/tests/integrationtests/indicators/test_attribute_completeness.py b/tests/integrationtests/indicators/test_attribute_completeness.py
@@ -8,6 +8,7 @@
 import pytest
 from approvaltests import verify
 
+from ohsome_quality_api.attributes.definitions import get_attributes
 from ohsome_quality_api.indicators.attribute_completeness.indicator import (
     AttributeCompleteness,
 )
@@ -243,3 +244,25 @@ def test_create_description_multiple_aggregation_types(
     indicator.absolute_value_2 = absolute_value_2
     indicator.create_description()
     assert aggregation in indicator.description
+
+
+def test_filters_match(topic_key_building_count, attribute_key_height):
+    indicator_attribute_keys = AttributeCompleteness(
+        get_topic_fixture(topic_key_building_count),
+        "foo",
+        attribute_keys=attribute_key_height,
+    )
+
+    attributes = get_attributes()
+    indicator_attribute_filter = AttributeCompleteness(
+        get_topic_fixture(topic_key_building_count),
+        "foo",
+        attribute_filter=attributes[topic_key_building_count][
+            attribute_key_height[0]
+        ].filter,
+    )
+
+    assert (
+        indicator_attribute_filter.attribute_filter
+        == indicator_attribute_keys.attribute_filter
+    )

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-__version__ = "1.8.1"`
	`1`	`+__version__ = "1.8.2"`
`2`	`2`	`__title__ = "ohsome quality API"`
`3`	`3`	`__description__ = "Data quality estimations for OpenStreetMap"`
`4`	`4`	`__author__ = "ohsome team"`
Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,11 @@ def __init__(`
`67`	`67`	`for k in self.attribute_keys`
`68`	`68`	`]`
`69`	`69`	`)`
	`70`	`+ else:`
	`71`	`+ self.attribute_filter = build_attribute_filter(`
	`72`	`+ self.attribute_filter,`
	`73`	`+ self.topic.key,`
	`74`	`+ )`
`70`	`75`
`71`	`76`	`async def preprocess(self) -> None:`
`72`	`77`	`# Get attribute filter`
Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 9: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.`
	`1`	`+Querying the ohsome API failed! Invalid filter syntax. Please look at the additional info and examples about the filter parameter at https://docs.ohsome.org/ohsome-api. Detailed error message: line 1, column 63: whitespaces, EQUALS (=), NOT_EQUALS (!=) or in expected, f encountered.`