semantic-link-labs 0.4.1__py3-none-any.whl → 0.5.0__py3-none-any.whl

sempy_labs/migration/_migration_validation.py

@@ -1,173 +1,10 @@
 import sempy
 import sempy.fabric as fabric
 import pandas as pd
-from sempy_labs._helper_functions import create_relationship_name
-from sempy_labs._tom import connect_semantic_model
 from typing import List, Optional, Union
+from sempy_labs._list_functions import list_semantic_model_objects
 from sempy._utils._log import log
 
-
-def list_semantic_model_objects(dataset: str, workspace: Optional[str] = None):
-    """
-    Shows a list of semantic model objects.
-
-    Parameters
-    ----------
-    dataset : str
-        Name of the semantic model.
-    workspace : str, default=None
-        The Fabric workspace name.
-        Defaults to None which resolves to the workspace of the attached lakehouse
-        or if no lakehouse attached, resolves to the workspace of the notebook.
-
-
-    Returns
-    -------
-    pandas.DataFrame
-        A pandas dataframe showing a list of objects in the semantic model
-    """
-
-    df = pd.DataFrame(columns=["Parent Name", "Object Name", "Object Type"])
-    with connect_semantic_model(
-        dataset=dataset, workspace=workspace, readonly=True
-    ) as tom:
-        for t in tom.model.Tables:
-            if t.CalculationGroup is not None:
-                new_data = {
-                    "Parent Name": t.Parent.Name,
-                    "Object Name": t.Name,
-                    "Object Type": "Calculation Group",
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-                for ci in t.CalculationGroup.CalculationItems:
-                    new_data = {
-                        "Parent Name": t.Name,
-                        "Object Name": ci.Name,
-                        "Object Type": str(ci.ObjectType),
-                    }
-                    df = pd.concat(
-                        [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                    )
-            elif any(str(p.SourceType) == "Calculated" for p in t.Partitions):
-                new_data = {
-                    "Parent Name": t.Parent.Name,
-                    "Object Name": t.Name,
-                    "Object Type": "Calculated Table",
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-            else:
-                new_data = {
-                    "Parent Name": t.Parent.Name,
-                    "Object Name": t.Name,
-                    "Object Type": str(t.ObjectType),
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-            for c in t.Columns:
-                if str(c.Type) != "RowNumber":
-                    if str(c.Type) == "Calculated":
-                        new_data = {
-                            "Parent Name": c.Parent.Name,
-                            "Object Name": c.Name,
-                            "Object Type": "Calculated Column",
-                        }
-                        df = pd.concat(
-                            [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                        )
-                    else:
-                        new_data = {
-                            "Parent Name": c.Parent.Name,
-                            "Object Name": c.Name,
-                            "Object Type": str(c.ObjectType),
-                        }
-                        df = pd.concat(
-                            [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                        )
-            for m in t.Measures:
-                new_data = {
-                    "Parent Name": m.Parent.Name,
-                    "Object Name": m.Name,
-                    "Object Type": str(m.ObjectType),
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-            for h in t.Hierarchies:
-                new_data = {
-                    "Parent Name": h.Parent.Name,
-                    "Object Name": h.Name,
-                    "Object Type": str(h.ObjectType),
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-                for l in h.Levels:
-                    new_data = {
-                        "Parent Name": l.Parent.Name,
-                        "Object Name": l.Name,
-                        "Object Type": str(l.ObjectType),
-                    }
-                    df = pd.concat(
-                        [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                    )
-            for p in t.Partitions:
-                new_data = {
-                    "Parent Name": p.Parent.Name,
-                    "Object Name": p.Name,
-                    "Object Type": str(p.ObjectType),
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-        for r in tom.model.Relationships:
-            rName = create_relationship_name(
-                r.FromTable.Name, r.FromColumn.Name, r.ToTable.Name, r.ToColumn.Name
-            )
-            new_data = {
-                "Parent Name": r.Parent.Name,
-                "Object Name": rName,
-                "Object Type": str(r.ObjectType),
-            }
-            df = pd.concat([df, pd.DataFrame(new_data, index=[0])], ignore_index=True)
-        for role in tom.model.Roles:
-            new_data = {
-                "Parent Name": role.Parent.Name,
-                "Object Name": role.Name,
-                "Object Type": str(role.ObjectType),
-            }
-            df = pd.concat([df, pd.DataFrame(new_data, index=[0])], ignore_index=True)
-            for rls in role.TablePermissions:
-                new_data = {
-                    "Parent Name": role.Name,
-                    "Object Name": rls.Name,
-                    "Object Type": str(rls.ObjectType),
-                }
-                df = pd.concat(
-                    [df, pd.DataFrame(new_data, index=[0])], ignore_index=True
-                )
-        for tr in tom.model.Cultures:
-            new_data = {
-                "Parent Name": tr.Parent.Name,
-                "Object Name": tr.Name,
-                "Object Type": str(tr.ObjectType),
-            }
-            df = pd.concat([df, pd.DataFrame(new_data, index=[0])], ignore_index=True)
-        for per in tom.model.Perspectives:
-            new_data = {
-                "Parent Name": per.Parent.Name,
-                "Object Name": per.Name,
-                "Object Type": str(per.ObjectType),
-            }
-            df = pd.concat([df, pd.DataFrame(new_data, index=[0])], ignore_index=True)
-
-    return df
-
-
 @log
 def migration_validation(
     dataset: str,
@@ -199,6 +36,10 @@ def migration_validation(
        A pandas dataframe showing a list of objects and whether they were successfully migrated. Also shows the % of objects which were migrated successfully.
     """
 
+    workspace = fabric.resolve_workspace_name(workspace)
+    if new_dataset_workspace is None:
+        new_dataset_workspace = workspace
+
     dfA = list_semantic_model_objects(dataset=dataset, workspace=workspace)
     dfB = list_semantic_model_objects(
         dataset=new_dataset, workspace=new_dataset_workspace

sempy_labs/migration/_refresh_calc_tables.py

@@ -3,7 +3,7 @@ import sempy.fabric as fabric
 import pandas as pd
 import re, datetime, time
 from pyspark.sql import SparkSession
-from sempy_labs._tom import connect_semantic_model
+from sempy_labs.tom import connect_semantic_model
 from typing import List, Optional, Union
 from sempy._utils._log import log
 import sempy_labs._icons as icons
@@ -30,6 +30,8 @@ def refresh_calc_tables(dataset: str, workspace: Optional[str] = None):
     timeout = datetime.timedelta(minutes=1)
     success = False
 
+    workspace = fabric.resolve_workspace_name(workspace)
+
     while not success:
         try:
             with connect_semantic_model(
@@ -118,10 +120,8 @@ def refresh_calc_tables(dataset: str, workspace: Optional[str] = None):
                             print(
                                 f"{icons.green_dot} Calculated table '{tName}' has been refreshed as the '{delta_table_name.lower()}' table in the lakehouse."
                             )
-                        except:
-                            print(
-                                f"{icons.red_dot} Failed to create calculated table '{tName}' as a delta table in the lakehouse."
-                            )
+                        except Exception as e:
+                            raise ValueError(f"{icons.red_dot} Failed to create calculated table '{tName}' as a delta table in the lakehouse.") from e
 
         except Exception as e:
             if datetime.datetime.now() - start_time > timeout:

sempy_labs/report/__init__.py

@@ -1,6 +1,6 @@
 from sempy_labs.report._generate_report import (
     create_report_from_reportjson,
-    update_report_from_reportjson,
+    #update_report_from_reportjson,
 )
 from sempy_labs.report._report_functions import (
     get_report_json,
@@ -20,7 +20,7 @@ from sempy_labs.report._report_rebind import (
 
 __all__ = [
     "create_report_from_reportjson",
-    "update_report_from_reportjson",
+    #"update_report_from_reportjson",
     "get_report_json",
     # report_dependency_tree,
     "export_report",

sempy_labs/report/_generate_report.py

@@ -4,13 +4,14 @@ import pandas as pd
 import json, base64, time
 from typing import Optional
 from sempy_labs._helper_functions import resolve_workspace_name_and_id
+import sempy_labs._icons as icons
 
 
 def create_report_from_reportjson(
     report: str,
     dataset: str,
-    report_json: str,
-    theme_json: Optional[str] = None,
+    report_json: dict,
+    theme_json: Optional[dict] = None,
     workspace: Optional[str] = None,
 ):
     """
@@ -22,9 +23,9 @@ def create_report_from_reportjson(
         Name of the report.
     dataset : str
         Name of the semantic model to connect to the report.
-    report_json : str
+    report_json : dict
         The report.json file to be used to create the report.
-    theme_json : str, default=None
+    theme_json : dict, default=None
         The theme.json file to be used for the theme of the report.
     workspace : str, default=None
         The Fabric workspace name.
@@ -40,10 +41,7 @@ def create_report_from_reportjson(
     dfI_model = dfI_m[(dfI_m["Display Name"] == dataset)]
 
     if len(dfI_model) == 0:
-        print(
-            f"ERROR: The '{dataset}' semantic model does not exist in the '{workspace}' workspace."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{dataset}' semantic model does not exist in the '{workspace}' workspace.")
 
     datasetId = dfI_model["Id"].iloc[0]
 
@@ -52,7 +50,7 @@ def create_report_from_reportjson(
 
     if len(dfI_rpt) > 0:
         print(
-            f"WARNING: '{report}' already exists as a report in the '{workspace}' workspace."
+            f"{icons.yellow_dot} '{report}' already exists as a report in the '{workspace}' workspace."
         )
         return
 
@@ -82,7 +80,7 @@ def create_report_from_reportjson(
     definitionPBIR = conv_b64(defPBIR)
     payloadReportJson = conv_b64(report_json)
 
-    if theme_json == None:
+    if theme_json is None:
         request_body = {
             "displayName": report,
             "type": objectType,
@@ -132,7 +130,7 @@ def create_report_from_reportjson(
     response = client.post(f"/v1/workspaces/{workspace_id}/items", json=request_body)
 
     if response.status_code == 201:
-        print("Report creation succeeded")
+        print(f"{icons.green_dot} Report creation succeeded")
         print(response.json())
     elif response.status_code == 202:
         operationId = response.headers["x-ms-operation-id"]
@@ -143,7 +141,7 @@ def create_report_from_reportjson(
             response = client.get(f"/v1/operations/{operationId}")
             response_body = json.loads(response.content)
         response = client.get(f"/v1/operations/{operationId}/result")
-        print("Report creation succeeded")
+        print(f"{icons.green_dot} Report creation succeeded")
         print(response.json())
 
 
@@ -167,14 +165,11 @@ def update_report_from_reportjson(
 
     (workspace, workspace_id) = resolve_workspace_name_and_id(workspace)
 
-    objectType = "Report"
-
     dfR = fabric.list_reports(workspace=workspace)
     dfR_filt = dfR[(dfR["Name"] == report) & (dfR["Report Type"] == "PowerBIReport")]
 
     if len(dfR_filt) == 0:
-        print(f"The '{report}' report does not exist in the '{workspace}' workspace.")
-        return
+        raise ValueError(f"{icons.red_dot} The '{report}' report does not exist in the '{workspace}' workspace.")
 
     reportId = dfR_filt["Id"].iloc[0]
     client = fabric.FabricRestClient()
@@ -215,7 +210,7 @@ def update_report_from_reportjson(
 
     request_body = {
         "displayName": report,
-        "type": objectType,
+        "type": 'Report',
         "definition": {
             "parts": [
                 {
@@ -238,7 +233,7 @@ def update_report_from_reportjson(
     )
 
     if response.status_code == 201:
-        print(f"The '{report}' report has been successfully updated.")
+        print(f"{icons.green_dot} The '{report}' report has been successfully updated.")
         # print(response.json())
     elif response.status_code == 202:
         operationId = response.headers["x-ms-operation-id"]
@@ -249,5 +244,5 @@ def update_report_from_reportjson(
             response = client.get(f"/v1/operations/{operationId}")
             response_body = json.loads(response.content)
         response = client.get(f"/v1/operations/{operationId}/result")
-        print(f"The '{report}' report has been successfully updated.")
+        print(f"{icons.green_dot} The '{report}' report has been successfully updated.")
         # print(response.json())

sempy_labs/report/_report_functions.py

@@ -1,7 +1,7 @@
 import sempy
 import sempy.fabric as fabric
 import pandas as pd
-import json, os, time, base64, copy, re
+import json, os, time, base64, copy
 from anytree import Node, RenderTree
 from powerbiclient import Report
 from synapse.ml.services import Translate
@@ -55,10 +55,7 @@ def get_report_json(
     dfI_filt = dfI[(dfI["Display Name"] == report)]
 
     if len(dfI_filt) == 0:
-        print(
-            f"{icons.red_dot} The '{report}' report does not exist in the '{workspace}' workspace."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{report}' report does not exist in the '{workspace}' workspace.")        
 
     itemId = dfI_filt["Id"].iloc[0]
     response = client.post(
@@ -73,11 +70,8 @@ def get_report_json(
 
     if save_to_file_name is not None:
         lakeAttach = lakehouse_attached()
-        if lakeAttach == False:
-            print(
-                f"{icons.red_dot} In order to save the report.json file, a lakehouse must be attached to the notebook. Please attach a lakehouse to this notebook."
-            )
-            return
+        if lakeAttach is False:
+            raise ValueError(f"{icons.red_dot} In order to save the report.json file, a lakehouse must be attached to the notebook. Please attach a lakehouse to this notebook.")
 
         lakehouse_id = fabric.get_lakehouse_id()
         lakehouse = resolve_lakehouse_name(lakehouse_id, workspace)
@@ -107,7 +101,7 @@ def report_dependency_tree(workspace: Optional[str] = None):
         or if no lakehouse attached, resolves to the workspace of the notebook.
     """
 
-    if workspace == None:
+    if workspace is None:
         workspaceId = fabric.get_workspace_id()
         workspace = fabric.resolve_workspace_name(workspaceId)
 
@@ -190,11 +184,8 @@ def export_report(
 
     lakeAttach = lakehouse_attached()
 
-    if lakeAttach == False:
-        print(
-            f"{icons.red_dot} In order to run the 'export_report' function, a lakehouse must be attached to the notebook. Please attach a lakehouse to this notebook."
-        )
-        return
+    if lakeAttach is False:
+        raise ValueError(f"{icons.red_dot} In order to run the 'export_report' function, a lakehouse must be attached to the notebook. Please attach a lakehouse to this notebook.")
 
     (workspace, workspace_id) = resolve_workspace_name_and_id(workspace)
 
@@ -204,15 +195,10 @@ def export_report(
         visual_name = [visual_name]
 
     if bookmark_name is not None and (page_name is not None or visual_name is not None):
-        print(
-            f"{icons.red_dot} If the 'bookmark_name' parameter is set, the 'page_name' and 'visual_name' parameters must not be set."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} If the 'bookmark_name' parameter is set, the 'page_name' and 'visual_name' parameters must not be set.")
+
     if visual_name is not None and page_name is None:
-        print(
-            f"{icons.red_dot} If the 'visual_name' parameter is set, the 'page_name' parameter must be set."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} If the 'visual_name' parameter is set, the 'page_name' parameter must be set.")
 
     validFormats = {
         "ACCESSIBLEPDF": ".pdf",
@@ -235,12 +221,9 @@ def export_report(
 
     fileExt = validFormats.get(export_format)
     if fileExt is None:
-        print(
-            f"{icons.red_dot} The '{export_format}' format is not a valid format for exporting Power BI reports. Please enter a valid format. Options: {validFormats}"
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{export_format}' format is not a valid format for exporting Power BI reports. Please enter a valid format. Options: {validFormats}")
 
-    if file_name == None:
+    if file_name is None:
         file_name = report + fileExt
     else:
         file_name = file_name + fileExt
@@ -255,10 +238,7 @@ def export_report(
     ]
 
     if len(dfI_filt) == 0:
-        print(
-            f"{icons.red_dot} The '{report}' report does not exist in the '{workspace}' workspace."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{report}' report does not exist in the '{workspace}' workspace.")
 
     reportType = dfI_filt["Type"].iloc[0]
 
@@ -279,23 +259,15 @@ def export_report(
     ]
 
     if reportType == "Report" and export_format in paginatedOnly:
-        print(
-            f"{icons.red_dot} The '{export_format}' format is only supported for paginated reports."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{export_format}' format is only supported for paginated reports.")
+
     if reportType == "PaginatedReport" and export_format in pbiOnly:
-        print(
-            f"{icons.red_dot} The '{export_format}' format is only supported for Power BI reports."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{export_format}' format is only supported for Power BI reports.")
 
     if reportType == "PaginatedReport" and (
         bookmark_name is not None or page_name is not None or visual_name is not None
     ):
-        print(
-            f"{icons.red_dot} Export for paginated reports does not support bookmarks/pages/visuals. Those parameters must not be set for paginated reports."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} Export for paginated reports does not support bookmarks/pages/visuals. Those parameters must not be set for paginated reports.")
 
     reportId = dfI_filt["Id"].iloc[0]
     client = fabric.PowerBIRestClient()
@@ -332,19 +304,15 @@ def export_report(
             for page in page_name:
                 dfPage_filt = dfPage[dfPage["Page ID"] == page]
                 if len(dfPage_filt) == 0:
-                    print(
-                        f"{icons.red_dot} The '{page}' page does not exist in the '{report}' report within the '{workspace}' workspace."
-                    )
-                    return
+                    raise ValueError(f"{icons.red_dot} The '{page}' page does not exist in the '{report}' report within the '{workspace}' workspace.")
+
                 page_dict = {"pageName": page}
                 request_body["powerBIReportConfiguration"]["pages"].append(page_dict)
 
     elif page_name is not None and visual_name is not None:
         if len(page_name) != len(visual_name):
-            print(
-                f"{icons.red_dot} Each 'visual_name' must map to a single 'page_name'."
-            )
-            return
+            raise ValueError(f"{icons.red_dot} Each 'visual_name' must map to a single 'page_name'.")
+
         if reportType == "Report":
             request_body = {"format": export_format, "powerBIReportConfiguration": {}}
 
@@ -356,10 +324,8 @@ def export_report(
                     (dfVisual["Page ID"] == page) & (dfVisual["Visual ID"] == visual)
                 ]
                 if len(dfVisual_filt) == 0:
-                    print(
-                        f"{icons.red_dot} The '{visual}' visual does not exist on the '{page}' in the '{report}' report within the '{workspace}' workspace."
-                    )
-                    return
+                    raise ValueError(f"{icons.red_dot} The '{visual}' visual does not exist on the '{page}' in the '{report}' report within the '{workspace}' workspace.")
+
                 page_dict = {"pageName": page, "visualName": visual}
                 request_body["powerBIReportConfiguration"]["pages"].append(page_dict)
                 a += 1
@@ -393,9 +359,7 @@ def export_report(
             )
             response_body = json.loads(response.content)
         if response_body["status"] == "Failed":
-            print(
-                f"{icons.red_dot} The export for the '{report}' report within the '{workspace}' workspace in the '{export_format}' format has failed."
-            )
+            raise ValueError(f"{icons.red_dot} The export for the '{report}' report within the '{workspace}' workspace in the '{export_format}' format has failed.")
         else:
             response = client.get(
                 f"/v1.0/myorg/groups/{workspace_id}/reports/{reportId}/exports/{exportId}/file"
@@ -447,10 +411,7 @@ def clone_report(
     dfI_filt = dfI[(dfI["Display Name"] == report)]
 
     if len(dfI_filt) == 0:
-        print(
-            f"{icons.red_dot} The '{report}' report does not exist within the '{workspace}' workspace."
-        )
-        return
+        raise ValueError(f"{icons.red_dot} The '{report}' report does not exist within the '{workspace}' workspace.")
 
     reportId = resolve_report_id(report, workspace)
 
@@ -462,11 +423,11 @@ def clone_report(
         dfW_filt = dfW[dfW["Name"] == target_workspace]
 
         if len(dfW_filt) == 0:
-            print(f"{icons.red_dot} The '{workspace}' is not a valid workspace.")
-            return
+            raise ValueError(f"{icons.red_dot} The '{workspace}' is not a valid workspace.")
+
         target_workspace_id = dfW_filt["Id"].iloc[0]
 
-    if target_dataset == None:
+    if target_dataset is None:
         dfR = fabric.list_reports(workspace=target_workspace)
         dfR_filt = dfR[dfR["Name"] == report]
         target_dataset_id = dfR_filt["Dataset Id"].iloc[0]
@@ -478,10 +439,8 @@ def clone_report(
         dfD_filt = dfD[dfD["Dataset Name"] == target_dataset]
 
         if len(dfD_filt) == 0:
-            print(
-                f"{icons.red_dot} The '{target_dataset}' target dataset does not exist in the '{target_workspace}' workspace."
-            )
-            return
+            raise ValueError(f"{icons.red_dot} The '{target_dataset}' target dataset does not exist in the '{target_workspace}' workspace.")
+
         target_dataset_id = dfD_filt["Dataset Id"].iloc[0]
 
     client = fabric.PowerBIRestClient()
@@ -508,9 +467,7 @@ def clone_report(
             f"{icons.green_dot} The '{report}' report has been successfully cloned as the '{cloned_report}' report within the '{target_workspace}' workspace using the '{target_dataset}' semantic model."
         )
     else:
-        print(
-            f"{icons.red_dot} POST request failed with status code: {response.status_code}"
-        )
+        raise ValueError(f"{icons.red_dot} POST request failed with status code: {response.status_code}")
 
 
 def launch_report(report: str, workspace: Optional[str] = None):
@@ -532,7 +489,7 @@ def launch_report(report: str, workspace: Optional[str] = None):
         An embedded Power BI report within the notebook.
     """
 
-    from .HelperFunctions import resolve_report_id
+    from sempy_labs import resolve_report_id
 
     (workspace, workspace_id) = resolve_workspace_name_and_id(workspace)
 
@@ -562,7 +519,7 @@ def list_report_pages(report: str, workspace: Optional[str] = None):
         A pandas dataframe showing the pages within a Power BI report and their properties.
     """
 
-    if workspace == None:
+    if workspace is None:
         workspace_id = fabric.get_workspace_id()
         workspace = fabric.resolve_workspace_name(workspace_id)
 
@@ -573,14 +530,14 @@ def list_report_pages(report: str, workspace: Optional[str] = None):
     reportJson = get_report_json(report=report, workspace=workspace)
 
     for section in reportJson["sections"]:
-        pageID = section["name"]
-        pageName = section["displayName"]
+        pageID = section.get("name")
+        pageName = section.get("displayName")
         # pageFilters = section['filters']
-        pageWidth = section["width"]
-        pageHeight = section["height"]
+        pageWidth = section.get("width")
+        pageHeight = section.get("height")
         visualCount = len(section["visualContainers"])
         pageHidden = False
-        pageConfig = section["config"]
+        pageConfig = section.get("config")
         pageConfigJson = json.loads(pageConfig)
 
         try:
@@ -626,7 +583,7 @@ def list_report_visuals(report: str, workspace: Optional[str] = None):
         A pandas dataframe showing the visuals within a Power BI report and their properties.
     """
 
-    if workspace == None:
+    if workspace is None:
         workspace_id = fabric.get_workspace_id()
         workspace = fabric.resolve_workspace_name(workspace_id)
 
@@ -681,7 +638,7 @@ def list_report_bookmarks(report: str, workspace: Optional[str] = None):
         A pandas dataframe showing the bookmarks within a Power BI report and their properties.
     """
 
-    if workspace == None:
+    if workspace is None:
         workspace_id = fabric.get_workspace_id()
         workspace = fabric.resolve_workspace_name(workspace_id)
 
@@ -750,6 +707,7 @@ def list_report_bookmarks(report: str, workspace: Optional[str] = None):
         )
 
 
+@log
 def translate_report_titles(
     report: str, languages: Union[str, List[str]], workspace: Optional[str] = None
 ):