findly.unified-reporting-sdk 0.7.9__tar.gz → 0.7.11__tar.gz

{findly_unified_reporting_sdk-0.7.9 → findly_unified_reporting_sdk-0.7.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: findly.unified-reporting-sdk
-Version: 0.7.9
+Version: 0.7.11
 Summary: 
 License: GPL-3.0-only
 Requires-Python: >=3.9,<4.0

{findly_unified_reporting_sdk-0.7.9 → findly_unified_reporting_sdk-0.7.11}/findly/unified_reporting_sdk/data_sources/common/common_parser.py

@@ -167,7 +167,7 @@ class CommonParser:
     def equalize_dataframe_rows(
         self, dataframes: List[pd.DataFrame], dimensions: List[str]
     ) -> List[pd.DataFrame]:
-        if len(dataframes) <= 1 or not dimensions:
+        if len(dataframes) <= 1 or not dimensions or all(df.empty for df in dataframes):
             return dataframes
 
         # Check if all dimensions exist in all dataframes

{findly_unified_reporting_sdk-0.7.9 → findly_unified_reporting_sdk-0.7.11}/findly/unified_reporting_sdk/data_sources/ga4/ga4_client.py

@@ -240,6 +240,18 @@ class GA4Client(ReportsClient):
             )
         return None
 
+    def _deduplicate_dimensions(self, dimensions: List[Dimension]) -> List[Dimension]:
+        """
+        Deduplicate a list of Dimension objects using their name as a unique key.
+        """
+        seen = set()
+        deduped = []
+        for d in dimensions:
+            if d.name not in seen:
+                seen.add(d.name)
+                deduped.append(d)
+        return deduped
+
     @backoff.on_exception(
         backoff.expo, Exception, max_time=60, max_tries=3, giveup=give_up
     )
@@ -260,10 +272,11 @@ class GA4Client(ReportsClient):
     ) -> Optional[RunReportResponse]:
         report_response = None
         try:
+            deduplicated_dimensions = self._deduplicate_dimensions(dimensions)
             request = RunReportRequest(
                 property=f"properties/{property_id}",
                 date_ranges=date_ranges,
-                dimensions=dimensions,
+                dimensions=deduplicated_dimensions,
                 metrics=metrics,
                 metric_filter=metric_filter,
                 dimension_filter=dimension_filter,
@@ -313,23 +326,48 @@ class GA4Client(ReportsClient):
         include_totals: bool = True,
         **kwargs: str,
     ) -> Optional[RunReportResponse]:
-        
-        # deduplicate dimensions, otherwise we get an error
-        deduplicated_dimensions = list(set(dimensions))
+        # Use the new deduplication method instead of set() to avoid the unhashable error.
+        deduplicated_dimensions = self._deduplicate_dimensions(dimensions)
 
-        return await self._decorated_query_from_parts(
-            metrics=metrics,
-            dimensions=deduplicated_dimensions,
-            date_ranges=date_ranges,
-            dimension_filter=dimension_filter,
-            metric_filter=metric_filter,
-            order_bys=order_bys,
-            limit=limit,
-            offset=offset,
-            property_id=property_id,
-            include_totals=include_totals,
-            **kwargs,
-        )
+        try:
+            request = RunReportRequest(
+                property=f"properties/{property_id}",
+                date_ranges=date_ranges,
+                dimensions=deduplicated_dimensions,
+                metrics=metrics,
+                metric_filter=metric_filter,
+                dimension_filter=dimension_filter,
+                order_bys=order_bys,
+                limit=limit,
+                offset=offset,
+                keep_empty_rows=True,
+                metric_aggregations=(
+                    [MetricAggregation.TOTAL] if include_totals else None
+                ),
+            )
+            report_response = await asyncio.wait_for(
+                self._client.run_report(request), timeout=REQUEST_TIMEOUT
+            )
+        except (TimeoutError, asyncio.exceptions.TimeoutError) as timeout_error:
+            LOGGER.warning(
+                {
+                    "error": str(timeout_error),
+                    "msg": "timeout_error_from_ga4_client",
+                    "property_id": property_id,
+                }
+            )
+            raise timeout_error
+        except Exception as e:
+            LOGGER.warning(
+                {
+                    "msg": "failed_to_run_report",
+                    "property_id": property_id,
+                    "error": str(e),
+                }
+            )
+            raise e
+
+        return report_response
 
     @cached(ttl=300, cache=Cache.MEMORY)  # Cache results for 5 minutes
     async def _decorated_query(

{findly_unified_reporting_sdk-0.7.9 → findly_unified_reporting_sdk-0.7.11}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "findly.unified-reporting-sdk"
-version = "0.7.9"
+version = "0.7.11"
 license = "GPL-3.0-only"
 description = ""
 authors = []