Update on "[ET] Add tsv_path to inspector_cli"

Differential Revision: [D66379005](https://our.internmc.facebook.com/intern/diff/D66379005/)

[ghstack-poisoned]

Author: jorgep31415

devtools/inspector/_inspector.py

@@ -1143,23 +1143,18 @@ def to_dataframe(
         ]
         return pd.concat(df_list, ignore_index=True)
 
-    def print_data_tabular(
+    def _prepare_dataframe(
         self,
-        file: IO[str] = sys.stdout,
         include_units: bool = True,
         include_delegate_debug_data: bool = False,
-    ) -> None:
+    ) -> pd.DataFrame:
         """
-        Displays the underlying EventBlocks in a structured tabular format, with each row representing an Event.
-
         Args:
-            file: Which IO stream to print to. Defaults to stdout.
-                Not used if this is in an IPython environment such as a Jupyter notebook.
             include_units: Whether headers should include units (default true)
             include_delegate_debug_data: Whether to include delegate debug metadata (default false)
 
         Returns:
-            None
+            Returns a pandas DataFrame of the Events in each EventBlock in the inspector, with additional filtering.
         """
         combined_df = self.to_dataframe(include_units, include_delegate_debug_data)
 
@@ -1171,7 +1166,29 @@ def print_data_tabular(
             ]
         filtered_column_df.reset_index(drop=True, inplace=True)
 
-        display_or_print_df(filtered_column_df, file)
+        return filtered_column_df
+
+    
+    def print_data_tabular(
+        self,
+        file: IO[str] = sys.stdout,
+        include_units: bool = True,
+        include_delegate_debug_data: bool = False,
+    ) -> None:
+        """
+        Displays the underlying EventBlocks in a structured tabular format, with each row representing an Event.
+
+        Args:
+            file: Which IO stream to print to. Defaults to stdout.
+                Not used if this is in an IPython environment such as a Jupyter notebook.
+            include_units: Whether headers should include units (default true)
+            include_delegate_debug_data: Whether to include delegate debug metadata (default false)
+
+        Returns:
+            None
+        """
+        df = self._prepare_dataframe(include_units, include_delegate_debug_data)
+        display_or_print_df(df, file)
 
 
     def save_data_to_tsv(
@@ -1187,17 +1204,8 @@ def save_data_to_tsv(
         Returns:
             None
         """
-        combined_df = self.to_dataframe()
-
-        # Filter out some columns and rows for better readability when printing
-        filtered_column_df = combined_df.drop(columns=EXCLUDED_COLUMNS_WHEN_PRINTING)
-        for filter_name in EXCLUDED_EVENTS_WHEN_PRINTING:
-            filtered_column_df = filtered_column_df[
-                ~filtered_column_df["event_name"].str.contains(filter_name)
-            ]
-        filtered_column_df.reset_index(drop=True, inplace=True)
-
-        filtered_column_df.to_csv(file, sep="\t")
+        df = self._prepare_dataframe()
+        df.to_csv(file, sep="\t")
 
 
     # TODO: write unit test