style: run ruff and fix line length

Author: matthiasschaub

ohsome_quality_api/indicators/attribute_completeness/indicator.py

@@ -197,11 +197,11 @@ def compute_units_for_all_and_matched(self):
             all = f"{int(self.absolute_value_1)} elements"
             matched = f"{int(self.absolute_value_2)} elements"
         elif self.topic.aggregation_type == "area":
-            all = f"{str(round(self.absolute_value_1/1000000, 2))} km²"
-            matched = f"{str(round(self.absolute_value_2/1000000, 2))} km²"
+            all = f"{str(round(self.absolute_value_1 / 1000000, 2))} km²"
+            matched = f"{str(round(self.absolute_value_2 / 1000000, 2))} km²"
         elif self.topic.aggregation_type == "length":
-            all = f"{str(round(self.absolute_value_1/1000, 2))} km"
-            matched = f"{str(round(self.absolute_value_2/1000, 2))} km"
+            all = f"{str(round(self.absolute_value_1 / 1000, 2))} km"
+            matched = f"{str(round(self.absolute_value_2 / 1000, 2))} km"
         else:
             raise ValueError("Invalid aggregation_type")
         return all, matched

ohsome_quality_api/indicators/road_comparison/indicator.py

@@ -121,8 +121,8 @@ def calculate(self) -> None:
             self.result.description += self.warnings[key] + "\n"
             self.result.description += (
                 f"{self.data_ref[key]['name']} has a road length of "
-                f"{(self.length_total[key]/1000):.2f} km, of which "
-                f"{(self.length_matched[key]/1000):.2f} km are covered by roads in "
+                f"{(self.length_total[key] / 1000):.2f} km, of which "
+                f"{(self.length_matched[key] / 1000):.2f} km are covered by roads in "
                 f"OSM. "
             )
 
@@ -169,6 +169,10 @@ def create_figure(self) -> None:
         for i, (name, ratio, date) in enumerate(
             zip(ref_name, ref_ratio, ref_processingdate)
         ):
+            hovertext = (
+                f"OSM Covered: {(self.length_matched[name] / 1000):.2f} km "
+                + f"({date:%b %d, %Y})"
+            )
             fig.add_trace(
                 pgo.Bar(
                     x=[name],
@@ -179,8 +183,7 @@ def create_figure(self) -> None:
                         line=dict(color=Color.GREY.value, width=1),
                     ),
                     width=0.4,
-                    hovertext=f"OSM Covered: {(self.length_matched[name]/1000):.2f} km"
-                    f" ({date:%b %d, %Y})",
+                    hovertext=hovertext,
                     hoverinfo="text",
                 )
             )

tests/integrationtests/indicators/test_road_comparison.py

@@ -84,8 +84,7 @@ def test_get_sources(self, topic_major_roads_length, feature_malta):
         indicator = RoadComparison(topic_major_roads_length, feature_malta)
         source = indicator.format_sources()
         assert (
-            "<a href='https://github.com/microsoft/RoadDetections'>"
-            "Microsoft Roads</a>"
+            "<a href='https://github.com/microsoft/RoadDetections'>Microsoft Roads</a>"
         ) in source
 
     def test_attribution(self, topic_major_roads_length, feature_malta):