semantic-link-labs 0.8.9__py3-none-any.whl → 0.8.11__py3-none-any.whl

sempy_labs/report/_report_rebind.py

@@ -1,20 +1,22 @@
 import sempy.fabric as fabric
 from sempy_labs._helper_functions import (
     resolve_dataset_id,
+    resolve_workspace_name_and_id,
     resolve_report_id,
 )
 from typing import Optional, List
 from sempy._utils._log import log
 import sempy_labs._icons as icons
 from sempy.fabric.exceptions import FabricHTTPException
+from uuid import UUID
 
 
 @log
 def report_rebind(
     report: str | List[str],
     dataset: str,
-    report_workspace: Optional[str] = None,
-    dataset_workspace: Optional[str] = None,
+    report_workspace: Optional[str | UUID] = None,
+    dataset_workspace: Optional[str | UUID] = None,
 ):
     """
     Rebinds a report to a semantic model.
@@ -27,23 +29,22 @@ def report_rebind(
         Name(s) of the Power BI report(s).
     dataset : str
         Name of the semantic model.
-    report_workspace : str, default=None
-        The name of the Fabric workspace in which the report resides.
+    report_workspace : str | uuid.UUID, default=None
+        The name or ID of the Fabric workspace in which the report resides.
         Defaults to None which resolves to the workspace of the attached lakehouse
         or if no lakehouse attached, resolves to the workspace of the notebook.
-    dataset_workspace : str, default=None
-        The name of the Fabric workspace in which the semantic model resides.
+    dataset_workspace : str | uuid.UUID, default=None
+        The name or ID of the Fabric workspace in which the semantic model resides.
         Defaults to None which resolves to the workspace of the attached lakehouse
         or if no lakehouse attached, resolves to the workspace of the notebook.
     """
 
-    if report_workspace is None:
-        report_workspace_id = fabric.get_workspace_id()
-        report_workspace = fabric.resolve_workspace_name(report_workspace_id)
-    else:
-        report_workspace_id = fabric.resolve_workspace_id(report_workspace)
+    (report_workspace_name, report_workspace_id) = resolve_workspace_name_and_id(
+        report_workspace
+    )
+
     if dataset_workspace is None:
-        dataset_workspace = report_workspace
+        dataset_workspace = report_workspace_name
 
     client = fabric.PowerBIRestClient()
 
@@ -51,14 +52,14 @@ def report_rebind(
         report = [report]
 
     for rpt in report:
-        reportId = resolve_report_id(report=rpt, workspace=report_workspace)
-        datasetId = resolve_dataset_id(dataset=dataset, workspace=dataset_workspace)
+        report_id = resolve_report_id(report=rpt, workspace=report_workspace_id)
+        dataset_id = resolve_dataset_id(dataset=dataset, workspace=dataset_workspace)
 
         # Prepare API
-        request_body = {"datasetId": datasetId}
+        request_body = {"datasetId": dataset_id}
 
         response = client.post(
-            f"/v1.0/myorg/groups/{report_workspace_id}/reports/{reportId}/Rebind",
+            f"/v1.0/myorg/groups/{report_workspace_id}/reports/{report_id}/Rebind",
             json=request_body,
         )
 

sempy_labs/report/_reportwrapper.py

@@ -8,6 +8,7 @@ from sempy_labs._helper_functions import (
     _add_part,
     lro,
     _decode_b64,
+    resolve_workspace_name_and_id,
 )
 from typing import Optional, List
 import pandas as pd
@@ -32,8 +33,8 @@ class ReportWrapper:
     ----------
     report : str
         The name of the report.
-    workspace : str
-        The name of the workspace in which the report resides.
+    workspace : str | uuid.UUID
+        The name or ID of the workspace in which the report resides.
         Defaults to None which resolves to the workspace of the attached lakehouse
         or if no lakehouse attached, resolves to the workspace of the notebook.
 
@@ -50,7 +51,7 @@ class ReportWrapper:
     def __init__(
         self,
         report: str,
-        workspace: Optional[str] = None,
+        workspace: Optional[str | UUID] = None,
     ):
         """
         Connects to a Power BI report and retrieves its definition.
@@ -61,8 +62,8 @@ class ReportWrapper:
         ----------
         report : str
             The name of the report.
-        workspace : str
-            The name of the workspace in which the report resides.
+        workspace : str | UUID
+            The name or ID of the workspace in which the report resides.
             Defaults to None which resolves to the workspace of the attached lakehouse
             or if no lakehouse attached, resolves to the workspace of the notebook.
 
@@ -77,11 +78,12 @@ class ReportWrapper:
         warnings.simplefilter(action="ignore", category=FutureWarning)
 
         self._report = report
-        self._workspace = workspace
-        self._workspace_id = fabric.resolve_workspace_id(workspace)
-        self._report_id = resolve_report_id(report, workspace)
+        (self._workspace_name, self._workspace_id) = resolve_workspace_name_and_id(
+            workspace
+        )
+        self._report_id = resolve_report_id(report, self._workspace_id)
         self.rdef = get_report_definition(
-            report=self._report, workspace=self._workspace
+            report=self._report, workspace=self._workspace_id
         )
 
         if len(self.rdef[self.rdef["path"] == "definition/report.json"]) == 0:
@@ -95,12 +97,14 @@ class ReportWrapper:
 
         from sempy_labs.tom import connect_semantic_model
 
-        dataset_id, dataset_name, dataset_workspace_id, dataset_workspace = (
-            resolve_dataset_from_report(report=self._report, workspace=self._workspace)
+        dataset_id, dataset_name, dataset_workspace_id, dataset_workspace_name = (
+            resolve_dataset_from_report(
+                report=self._report, workspace=self._workspace_id
+            )
         )
 
         with connect_semantic_model(
-            dataset=dataset_name, readonly=True, workspace=dataset_workspace
+            dataset=dataset_id, readonly=True, workspace=dataset_workspace_id
         ) as tom:
             for index, row in dataframe.iterrows():
                 obj_type = row["Object Type"]
@@ -218,7 +222,7 @@ class ReportWrapper:
         theme_collection = rptJson.get("themeCollection", {})
         if theme_type not in theme_collection:
             raise ValueError(
-                f"{icons.red_dot} The {self._report} report within the '{self._workspace} workspace has no custom theme."
+                f"{icons.red_dot} The {self._report} report within the '{self._workspace_name} workspace has no custom theme."
             )
         ct = theme_collection.get(theme_type)
         theme_name = ct["name"]
@@ -413,7 +417,7 @@ class ReportWrapper:
                             )
 
         df["Page URL"] = df["Page Name"].apply(
-            lambda page_name: f"{helper.get_web_url(report=self._report, workspace=self._workspace)}/{page_name}"
+            lambda page_name: f"{helper.get_web_url(report=self._report, workspace=self._workspace_id)}/{page_name}"
         )
 
         bool_cols = ["Hidden", "Locked", "Used"]
@@ -691,7 +695,7 @@ class ReportWrapper:
         df[bool_cols] = df[bool_cols].astype(bool)
 
         df["Page URL"] = df["Page Name"].apply(
-            lambda page_name: f"{helper.get_web_url(report=self._report, workspace=self._workspace)}/{page_name}"
+            lambda page_name: f"{helper.get_web_url(report=self._report, workspace=self._workspace_id)}/{page_name}"
         )
 
         return df
@@ -1170,9 +1174,9 @@ class ReportWrapper:
         )
 
         if extended:
-            dataset_id, dataset_name, dataset_workspace_id, dataset_workspace = (
+            dataset_id, dataset_name, dataset_workspace_id, dataset_workspace_name = (
                 resolve_dataset_from_report(
-                    report=self._report, workspace=self._workspace
+                    report=self._report, workspace=self._workspace_id
                 )
             )
 
@@ -1195,7 +1199,7 @@ class ReportWrapper:
                 return object_validators.get(row["Object Type"], lambda: False)()
 
             with connect_semantic_model(
-                dataset=dataset_name, readonly=True, workspace=dataset_workspace
+                dataset=dataset_id, readonly=True, workspace=dataset_workspace_id
             ) as tom:
                 df["Valid Semantic Model Object"] = df.apply(
                     lambda row: check_validity(tom, row), axis=1
@@ -1214,11 +1218,13 @@ class ReportWrapper:
             .drop_duplicates()
             .reset_index(drop=True)
         )
-        dataset_id, dataset_name, dataset_workspace_id, dataset_workspace = (
-            resolve_dataset_from_report(report=self._report, workspace=self._workspace)
+        dataset_id, dataset_name, dataset_workspace_id, dataset_workspace_name = (
+            resolve_dataset_from_report(
+                report=self._report, workspace=self._workspace_id
+            )
         )
         dep = get_measure_dependencies(
-            dataset=dataset_name, workspace=dataset_workspace
+            dataset=dataset_id, workspace=dataset_workspace_id
         )
         rpt_measures = df[df["Object Type"] == "Measure"]["Object Name"].values
         new_rows = dep[dep["Object Name"].isin(rpt_measures)][
@@ -1232,7 +1238,7 @@ class ReportWrapper:
         )
 
         result_df["Dataset Name"] = dataset_name
-        result_df["Dataset Workspace Name"] = dataset_workspace
+        result_df["Dataset Workspace Name"] = dataset_workspace_name
         colName = "Dataset Name"
         result_df.insert(0, colName, result_df.pop(colName))
         colName = "Dataset Workspace Name"
@@ -1539,7 +1545,7 @@ class ReportWrapper:
 
         self.update_report(request_body=request_body)
         print(
-            f"{icons.green_dot} The '{theme_name}' theme has been set as the theme for the '{self._report}' report within the '{self._workspace}' workspace."
+            f"{icons.green_dot} The '{theme_name}' theme has been set as the theme for the '{self._report}' report within the '{self._workspace_name}' workspace."
         )
 
     def set_active_page(self, page_name: str):
@@ -1567,7 +1573,7 @@ class ReportWrapper:
         self._update_single_file(file_name=pages_file, new_payload=file_payload)
 
         print(
-            f"{icons.green_dot} The '{page_display_name}' page has been set as the active page in the '{self._report}' report within the '{self._workspace}' workspace."
+            f"{icons.green_dot} The '{page_display_name}' page has been set as the active page in the '{self._report}' report within the '{self._workspace_name}' workspace."
         )
 
     def set_page_type(self, page_name: str, page_type: str):
@@ -1640,7 +1646,7 @@ class ReportWrapper:
                 cv_remove_display.append(cv_display)
         if len(cv_remove) == 0:
             print(
-                f"{icons.green_dot} There are no unnecessary custom visuals in the '{self._report}' report within the '{self._workspace}' workspace."
+                f"{icons.green_dot} There are no unnecessary custom visuals in the '{self._report}' report within the '{self._workspace_name}' workspace."
             )
             return
 
@@ -1662,7 +1668,7 @@ class ReportWrapper:
 
         self.update_report(request_body=request_body)
         print(
-            f"{icons.green_dot} The {cv_remove_display} custom visuals have been removed from the '{self._report}' report within the '{self._workspace}' workspace."
+            f"{icons.green_dot} The {cv_remove_display} custom visuals have been removed from the '{self._report}' report within the '{self._workspace_name}' workspace."
         )
 
     def migrate_report_level_measures(self, measures: Optional[str | List[str]] = None):
@@ -1681,12 +1687,14 @@ class ReportWrapper:
         rlm = self.list_report_level_measures()
         if len(rlm) == 0:
             print(
-                f"{icons.green_dot} The '{self._report}' report within the '{self._workspace}' workspace has no report-level measures."
+                f"{icons.green_dot} The '{self._report}' report within the '{self._workspace_name}' workspace has no report-level measures."
             )
             return
 
-        dataset_id, dataset_name, dataset_workspace_id, dataset_workspace = (
-            resolve_dataset_from_report(report=self._report, workspace=self._workspace)
+        dataset_id, dataset_name, dataset_workspace_id, dataset_workspace_name = (
+            resolve_dataset_from_report(
+                report=self._report, workspace=self._workspace_id
+            )
         )
 
         if isinstance(measures, str):
@@ -1703,7 +1711,7 @@ class ReportWrapper:
 
         mCount = 0
         with connect_semantic_model(
-            dataset=dataset_name, readonly=False, workspace=dataset_workspace
+            dataset=dataset_id, readonly=False, workspace=dataset_workspace_id
         ) as tom:
             for _, r in rlm.iterrows():
                 tableName = r["Table Name"]
@@ -1748,7 +1756,7 @@ class ReportWrapper:
 
         self.update_report(request_body=request_body)
         print(
-            f"{icons.green_dot} The report-level measures have been migrated to the '{dataset_name}' semantic model within the '{dataset_workspace}' workspace."
+            f"{icons.green_dot} The report-level measures have been migrated to the '{dataset_name}' semantic model within the '{dataset_workspace_name}' workspace."
         )
 
     def set_page_visibility(self, page_name: str, hidden: bool):
@@ -1798,7 +1806,7 @@ class ReportWrapper:
 
         if len(dfP_filt) == 0:
             print(
-                f"{icons.green_dot} There are no Tooltip or Drillthrough pages in the '{self._report}' report within the '{self._workspace}' workspace."
+                f"{icons.green_dot} There are no Tooltip or Drillthrough pages in the '{self._report}' report within the '{self._workspace_name}' workspace."
             )
             return
 
@@ -1837,7 +1845,7 @@ class ReportWrapper:
 
         self.update_report(request_body=request_body)
         print(
-            f"{icons.green_dot} Show items with data has been disabled for all visuals in the '{self._report}' report within the '{self._workspace}' workspace."
+            f"{icons.green_dot} Show items with data has been disabled for all visuals in the '{self._report}' report within the '{self._workspace_name}' workspace."
         )
 
     # Set Annotations

sempy_labs/tom/_model.py

@@ -7,6 +7,8 @@ from sempy_labs._helper_functions import (
     format_dax_object_name,
     generate_guid,
     _make_list_unique,
+    resolve_dataset_name_and_id,
+    resolve_workspace_name_and_id,
 )
 from sempy_labs._list_functions import list_relationships
 from sempy_labs._refresh_semantic_model import refresh_semantic_model
@@ -17,6 +19,7 @@ from sempy._utils._log import log
 import sempy_labs._icons as icons
 from sempy.fabric.exceptions import FabricHTTPException
 import ast
+from uuid import UUID
 
 if TYPE_CHECKING:
     import Microsoft.AnalysisServices.Tabular
@@ -27,27 +30,33 @@ class TOMWrapper:
     """
     Convenience wrapper around the TOM object model for a semantic model. Always use the connect_semantic_model function to make sure the TOM object is initialized correctly.
 
-    `XMLA read/write endpoints <https://learn.microsoft.com/power-bi/enterprise/service-premium-connect-tools#to-enable-read-write-for-a-premium-capacity>`_ must
-     be enabled if setting the readonly parameter to False.
+    `XMLA read/write endpoints <https://learn.microsoft.com/power-bi/enterprise/service-premium-connect-tools#to-enable-read-write-for-a-premium-capacity>`_ must be enabled if setting the readonly parameter to False.
     """
 
-    _dataset: str
-    _workspace: str
+    _dataset_id: UUID
+    _dataset_name: str
+    _workspace_id: UUID
+    _workspace_name: str
     _readonly: bool
     _tables_added: List[str]
     _table_map = dict
     _column_map = dict
 
     def __init__(self, dataset, workspace, readonly):
-        self._dataset = dataset
-        self._workspace = workspace
+
+        (workspace_name, workspace_id) = resolve_workspace_name_and_id(workspace)
+        (dataset_name, dataset_id) = resolve_dataset_name_and_id(dataset, workspace_id)
+        self._dataset_id = dataset_id
+        self._dataset_name = dataset_name
+        self._workspace_name = workspace_name
+        self._workspace_id = workspace_id
         self._readonly = readonly
         self._tables_added = []
 
         self._tom_server = fabric.create_tom_server(
-            readonly=readonly, workspace=workspace
+            readonly=readonly, workspace=workspace_id
         )
-        self.model = self._tom_server.Databases.GetByName(dataset).Model
+        self.model = self._tom_server.Databases[dataset_id].Model
 
         self._table_map = {}
         self._column_map = {}
@@ -2160,7 +2169,9 @@ class TOMWrapper:
             )
             """
             df = fabric.evaluate_dax(
-                dataset=self._dataset, workspace=self._workspace, dax_string=dax_query
+                dataset=self._dataset_id,
+                workspace=self._workspace_id,
+                dax_string=dax_query,
             )
             value = df["[1]"].iloc[0]
             if value != "1":
@@ -2424,7 +2435,7 @@ class TOMWrapper:
             )
         except Exception:
             raise ValueError(
-                f"{icons.red_dot} The '{measure_name}' measure does not exist in the '{self._dataset}' semantic model within the '{self._workspace}'."
+                f"{icons.red_dot} The '{measure_name}' measure does not exist in the '{self._dataset_name}' semantic model within the '{self._workspace_name}'."
             )
 
         graphics = [
@@ -2467,7 +2478,7 @@ class TOMWrapper:
                 )
             except Exception:
                 raise ValueError(
-                    f"{icons.red_dot} The '{target}' measure does not exist in the '{self._dataset}' semantic model within the '{self._workspace}'."
+                    f"{icons.red_dot} The '{target}' measure does not exist in the '{self._dataset_name}' semantic model within the '{self._workspace_name}'."
                 )
 
         if measure_target:
@@ -2793,7 +2804,7 @@ class TOMWrapper:
                     success = True
             if not success:
                 raise ValueError(
-                    f"{icons.red_dot} The '{obj}' object was not found in the '{self._dataset}' semantic model."
+                    f"{icons.red_dot} The '{obj}' object was not found in the '{self._dataset_name}' semantic model."
                 )
             else:
                 i += 1
@@ -2881,19 +2892,19 @@ class TOMWrapper:
         from sempy_labs._list_functions import list_tables
 
         dfT = list_tables(
-            dataset=self._dataset, workspace=self._workspace, extended=True
+            dataset=self._dataset_id, workspace=self._workspace_id, extended=True
         )
         dfC = fabric.list_columns(
-            dataset=self._dataset, workspace=self._workspace, extended=True
+            dataset=self._dataset_id, workspace=self._workspace_id, extended=True
         )
         dfP = fabric.list_partitions(
-            dataset=self._dataset, workspace=self._workspace, extended=True
+            dataset=self._dataset_id, workspace=self._workspace_id, extended=True
         )
         dfH = fabric.list_hierarchies(
-            dataset=self._dataset, workspace=self._workspace, extended=True
+            dataset=self._dataset_id, workspace=self._workspace_id, extended=True
         )
         dfR = list_relationships(
-            dataset=self._dataset, workspace=self._workspace, extended=True
+            dataset=self._dataset_id, workspace=self._workspace_id, extended=True
         )
 
         for t in self.model.Tables:
@@ -3276,6 +3287,10 @@ class TOMWrapper:
         """
         import Microsoft.AnalysisServices.Tabular as TOM
 
+        dependencies = dependencies[
+            dependencies["Object Name"] == dependencies["Parent Node"]
+        ]
+
         for obj in self.depends_on(object=object, dependencies=dependencies):
             if obj.ObjectType == TOM.ObjectType.Measure:
                 if (f"{obj.Parent.Name}[{obj.Name}]" in object.Expression) or (
@@ -3302,12 +3317,16 @@ class TOMWrapper:
         """
         import Microsoft.AnalysisServices.Tabular as TOM
 
+        dependencies = dependencies[
+            dependencies["Object Name"] == dependencies["Parent Node"]
+        ]
+
         def create_pattern(tableList, b):
             patterns = [
-                r"(?<!" + re.escape(table) + r"\[)(?<!" + re.escape(table) + r"'\[)"
+                r"(?<!" + re.escape(table) + r")(?<!'" + re.escape(table) + r"')"
                 for table in tableList
             ]
-            combined_pattern = "".join(patterns) + re.escape(b)
+            combined_pattern = "".join(patterns) + re.escape(f"[{b}]")
             return combined_pattern
 
         for obj in self.depends_on(object=object, dependencies=dependencies):
@@ -3338,7 +3357,9 @@ class TOMWrapper:
         usingView = False
 
         if self.is_direct_lake():
-            df = check_fallback_reason(dataset=self._dataset, workspace=self._workspace)
+            df = check_fallback_reason(
+                dataset=self._dataset_id, workspace=self._workspace_id
+            )
             df_filt = df[df["FallbackReasonID"] == 2]
 
             if len(df_filt) > 0:
@@ -3385,7 +3406,7 @@ class TOMWrapper:
 
         if rp is None:
             print(
-                f"{icons.yellow_dot} The '{table_name}' table in the '{self._dataset}' semantic model within the '{self._workspace}' workspace does not have an incremental refresh policy."
+                f"{icons.yellow_dot} The '{table_name}' table in the '{self._dataset_name}' semantic model within the '{self._workspace_name}' workspace does not have an incremental refresh policy."
             )
         else:
             print(f"Table Name: {table_name}")
@@ -3884,14 +3905,14 @@ class TOMWrapper:
 
         if table_name is None:
             raise ValueError(
-                f"{icons.red_dot} The '{measure_name}' is not a valid measure in the '{self._dataset}' semantic model within the '{self._workspace}' workspace."
+                f"{icons.red_dot} The '{measure_name}' is not a valid measure in the '{self._dataset_name}' semantic model within the '{self._workspace_name}' workspace."
             )
 
         table_name = matching_measures[0]
         # Validate date table
         if not self.is_date_table(date_table):
             raise ValueError(
-                f"{icons.red_dot} The '{date_table}' table is not a valid date table in the '{self._dataset}' wemantic model within the '{self._workspace}' workspace."
+                f"{icons.red_dot} The '{date_table}' table is not a valid date table in the '{self._dataset_name}' wemantic model within the '{self._workspace_name}' workspace."
             )
 
         # Extract date key from date table
@@ -3903,7 +3924,7 @@ class TOMWrapper:
 
         if not matching_columns:
             raise ValueError(
-                f"{icons.red_dot} The '{date_table}' table does not have a date key column in the '{self._dataset}' semantic model within the '{self._workspace}' workspace."
+                f"{icons.red_dot} The '{date_table}' table does not have a date key column in the '{self._dataset_name}' semantic model within the '{self._workspace_name}' workspace."
             )
 
         date_key = matching_columns[0]
@@ -4383,7 +4404,6 @@ class TOMWrapper:
         if isinstance(measure_name, str):
             measure_name = [measure_name]
 
-        workspace_id = fabric.resolve_workspace_id(self._workspace)
         client = fabric.FabricRestClient()
 
         if len(measure_name) > max_batch_size:
@@ -4402,7 +4422,7 @@ class TOMWrapper:
                         "modelItems": [],
                     },
                 },
-                "workspaceId": workspace_id,
+                "workspaceId": self._workspace_id,
                 "artifactInfo": {"artifactType": "SemanticModel"},
             }
             for m_name in measure_list:
@@ -4413,7 +4433,7 @@ class TOMWrapper:
                 )
                 if t_name is None:
                     raise ValueError(
-                        f"{icons.red_dot} The '{m_name}' measure does not exist in the '{self._dataset}' semantic model within the '{self._workspace}' workspace."
+                        f"{icons.red_dot} The '{m_name}' measure does not exist in the '{self._dataset_name}' semantic model within the '{self._workspace_name}' workspace."
                     )
 
                 new_item = {
@@ -4515,6 +4535,69 @@ class TOMWrapper:
             TOM.ValueFilterBehaviorType, value_filter_behavior
         )
 
+    def add_role_member(self, role_name: str, member: str | List[str]):
+        """
+        Adds an external model role member (AzureAD) to a role.
+
+        Parameters
+        ----------
+        role_name : str
+            The role name.
+        member : str | List[str]
+            The email address(es) of the member(s) to add.
+        """
+
+        import Microsoft.AnalysisServices.Tabular as TOM
+
+        if isinstance(member, str):
+            member = [member]
+
+        role = self.model.Roles[role_name]
+        current_members = [m.MemberName for m in role.Members]
+
+        for m in member:
+            if m not in current_members:
+                rm = TOM.ExternalModelRoleMember()
+                rm.IdentityProvider = "AzureAD"
+                rm.MemberName = m
+                role.Members.Add(rm)
+                print(
+                    f"{icons.green_dot} '{m}' has been added as a member of the '{role_name}' role."
+                )
+            else:
+                print(
+                    f"{icons.yellow_dot} '{m}' is already a member in the '{role_name}' role."
+                )
+
+    def remove_role_member(self, role_name: str, member: str | List[str]):
+        """
+        Removes an external model role member (AzureAD) from a role.
+
+        Parameters
+        ----------
+        role_name : str
+            The role name.
+        member : str | List[str]
+            The email address(es) of the member(s) to remove.
+        """
+
+        if isinstance(member, str):
+            member = [member]
+
+        role = self.model.Roles[role_name]
+        current_members = {m.MemberName: m.Name for m in role.Members}
+        for m in member:
+            name = current_members.get(m)
+            if name is not None:
+                role.Members.Remove(role.Members[name])
+                print(
+                    f"{icons.green_dot} The '{m}' member has been removed from the '{role_name}' role."
+                )
+            else:
+                print(
+                    f"{icons.yellow_dot} '{m}' is not a member of the '{role_name}' role."
+                )
+
     def close(self):
 
         if not self._readonly and self.model is not None:
@@ -4572,9 +4655,9 @@ class TOMWrapper:
 
             if len(self._tables_added) > 0:
                 refresh_semantic_model(
-                    dataset=self._dataset,
+                    dataset=self._dataset_id,
                     tables=self._tables_added,
-                    workspace=self._workspace,
+                    workspace=self._workspace_id,
                 )
             self.model = None
 
@@ -4584,19 +4667,19 @@ class TOMWrapper:
 @log
 @contextmanager
 def connect_semantic_model(
-    dataset: str, readonly: bool = True, workspace: Optional[str] = None
+    dataset: str | UUID, readonly: bool = True, workspace: Optional[str] = None
 ) -> Iterator[TOMWrapper]:
     """
     Connects to the Tabular Object Model (TOM) within a semantic model.
 
     Parameters
     ----------
-    dataset : str
-        Name of the semantic model.
+    dataset : str | uuid.UUID
+        Name or ID of the semantic model.
     readonly: bool, default=True
         Whether the connection is read-only or read/write. Setting this to False enables read/write which saves the changes made back to the server.
-    workspace : str, default=None
-        The Fabric workspace name.
+    workspace : str | uuid.UUID, default=None
+        The Fabric workspace name or ID.
         Defaults to None which resolves to the workspace of the attached lakehouse
         or if no lakehouse attached, resolves to the workspace of the notebook.
 
@@ -4609,10 +4692,6 @@ def connect_semantic_model(
     # initialize .NET to make sure System and Microsoft.AnalysisServices.Tabular is defined
     sempy.fabric._client._utils._init_analysis_services()
 
-    if workspace is None:
-        workspace_id = fabric.get_workspace_id()
-        workspace = fabric.resolve_workspace_name(workspace_id)
-
     tw = TOMWrapper(dataset=dataset, workspace=workspace, readonly=readonly)
     try:
         yield tw