Internal change

PiperOrigin-RevId: 705582910
tensorflow · Dec 12, 2024 · 4d0a1d4 · 4d0a1d4
1 parent bfa4df9
commit 4d0a1d4
Show file tree

Hide file tree

Showing 71 changed files with 106 additions and 31 deletions.
diff --git a/defs/BUILD.bazel → defs/BUILD b/defs/BUILD.bazel → defs/BUILD
diff --git a/frontend/app/BUILD b/frontend/app/BUILD
@@ -35,5 +35,6 @@ xprof_ng_module(
 sass_binary(
     name = "app_css",
     src = "app.scss",
+    # stack = False,
     sourcemap = False,
 )
diff --git a/frontend/app/components/capture_profile/BUILD b/frontend/app/components/capture_profile/BUILD
@@ -33,6 +33,7 @@ xprof_ng_module(
 sass_binary(
     name = "capture_profile_css",
     src = "capture_profile.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/capture_profile/capture_profile_dialog/BUILD b/frontend/app/components/capture_profile/capture_profile_dialog/BUILD
@@ -33,6 +33,7 @@ xprof_ng_module(
 sass_binary(
     name = "capture_profile_dialog_css",
     src = "capture_profile_dialog.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/chart/default_data_provider.ts b/frontend/app/components/chart/default_data_provider.ts
@@ -79,7 +79,7 @@ export class DefaultDataProvider implements ChartDataProvider {
 
 /** A chart data provider that accepts array data. */
 export class ArrayDataProvider extends DefaultDataProvider {
-   parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
+  override parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
     if (data) {
       /* tslint:disable no-any */
       this.dataTable = google.visualization.arrayToDataTable(data as any[]);
@@ -101,7 +101,7 @@ export class ReplicaGroupDataProvider extends DefaultDataProvider {
   hloOpNameIndex?: number;   // 'hlo_op_expression' column
   selfTimeIndex?: number;    // 'total_self_time' column
 
-   parseData(data: SimpleDataTable) {
+  override parseData(data: SimpleDataTable) {
     const rowWithReplicaGroups: google.visualization.DataObjectRow[] = [];
 
     if (!data || !data.cols || !data.rows) return;

diff --git a/frontend/app/components/chart/org_chart/BUILD b/frontend/app/components/chart/org_chart/BUILD
@@ -22,5 +22,6 @@ xprof_ng_module(
 sass_binary(
     name = "org_chart_css",
     src = "org_chart.scss",
+    # stack = False,
     sourcemap = False,
 )
diff --git a/frontend/app/components/chart/table/BUILD b/frontend/app/components/chart/table/BUILD
@@ -25,5 +25,6 @@ xprof_ng_module(
 sass_binary(
     name = "table_css",
     src = "table.scss",
+    # stack = False,
     sourcemap = False,
 )
diff --git a/frontend/app/components/controls/category_filter/BUILD b/frontend/app/components/controls/category_filter/BUILD
@@ -26,5 +26,6 @@ xprof_ng_module(
 sass_binary(
     name = "category_filter_css",
     src = "category_filter.scss",
+    # stack = False,
     sourcemap = False,
 )
diff --git a/frontend/app/components/controls/export_as_csv/BUILD b/frontend/app/components/controls/export_as_csv/BUILD
@@ -25,6 +25,7 @@ xprof_ng_module(
 sass_binary(
     name = "export_as_csv_css",
     src = "export_as_csv.scss",
+    # stack = True,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/controls/string_filter/BUILD b/frontend/app/components/controls/string_filter/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         "string_filter.ng.html",
         "string_filter_css",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/common",
         "@npm//@angular/core",
@@ -26,6 +27,7 @@ xprof_ng_module(
 sass_binary(
     name = "string_filter_css",
     src = "string_filter.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/dcn_collective_stats/BUILD b/frontend/app/components/dcn_collective_stats/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":dcn_collective_stats_css",
         "dcn_collective_stats.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/common",
         "@npm//@angular/core",
@@ -35,6 +36,7 @@ xprof_ng_module(
 sass_binary(
     name = "dcn_collective_stats_css",
     src = "dcn_collective_stats.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/diagnostics_view/BUILD b/frontend/app/components/diagnostics_view/BUILD
@@ -22,6 +22,7 @@ xprof_ng_module(
 sass_binary(
     name = "diagnostics_view_css",
     src = "diagnostics_view.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/empty_page/BUILD b/frontend/app/components/empty_page/BUILD
@@ -23,6 +23,7 @@ xprof_ng_module(
 sass_binary(
     name = "empty_page_css",
     src = "empty_page.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/framework_op_stats/BUILD b/frontend/app/components/framework_op_stats/BUILD
@@ -31,6 +31,7 @@ xprof_ng_module(
         ":framework_op_stats_css",
         "framework_op_stats.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/core",
         "@npm//@ngrx/store",
@@ -58,6 +59,7 @@ xprof_ng_module(
 sass_binary(
     name = "framework_op_stats_css",
     src = "framework_op_stats.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/framework_op_stats/flop_rate_chart/BUILD b/frontend/app/components/framework_op_stats/flop_rate_chart/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":flop_rate_chart_css",
         "flop_rate_chart.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/core",
         "@org_xprof//frontend/app/common/interfaces",
@@ -27,5 +28,6 @@ xprof_ng_module(
 sass_binary(
     name = "flop_rate_chart_css",
     src = "flop_rate_chart.scss",
+    # stack = False,
     sourcemap = False,
 )
diff --git a/frontend/app/components/framework_op_stats/model_properties/BUILD b/frontend/app/components/framework_op_stats/model_properties/BUILD
@@ -22,5 +22,6 @@ xprof_ng_module(
 sass_binary(
     name = "model_properties_css",
     src = "model_properties.scss",
+    # stack = False,
     sourcemap = False,
 )
diff --git a/frontend/app/components/framework_op_stats/operations_table/BUILD b/frontend/app/components/framework_op_stats/operations_table/BUILD
@@ -14,6 +14,7 @@ xprof_ng_module(
         ":operations_table_css",
         "operations_table.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         ":operations_table_data_provider",
         "@npm//@angular/core",
@@ -42,6 +43,7 @@ ts_library(
 sass_binary(
     name = "operations_table_css",
     src = "operations_table.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/...tend/app/components/framework_op_stats/operations_table/operations_table_data_provider.ts b/...tend/app/components/framework_op_stats/operations_table/operations_table_data_provider.ts
@@ -27,7 +27,7 @@ export class OperationsTableDataProvider extends DefaultDataProvider {
         diffData ? new google.visualization.DataTable(diffData) : undefined;
   }
 
-   process(): google.visualization.DataTable|google.visualization.DataView|null {
+  override process(): google.visualization.DataTable|google.visualization.DataView|null {
     if (!this.dataTable ||
         (this.opExecutor !== OpExecutor.DEVICE &&
          this.opExecutor !== OpExecutor.HOST)) {
@@ -115,7 +115,7 @@ export class OperationsTableDataProvider extends DefaultDataProvider {
     return dataView;
   }
 
-   getOptions(): ChartOptions|null {
+  override getOptions(): ChartOptions|null {
     return this.options;
   }
 

diff --git a/frontend/app/components/framework_op_stats/stats_table/BUILD b/frontend/app/components/framework_op_stats/stats_table/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":stats_table_css",
         "stats_table.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         ":stats_table_data_provider",
         "@npm//@angular/core",
@@ -42,6 +43,7 @@ xprof_ng_module(
 sass_binary(
     name = "stats_table_css",
     src = "stats_table.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/framework_op_stats/stats_table/stats_table_data_provider.ts b/frontend/app/components/framework_op_stats/stats_table/stats_table_data_provider.ts
@@ -42,7 +42,7 @@ export class StatsTableDataProvider extends DefaultDataProvider {
 
   private readonly totalOperationsChanged = new EventEmitter<string>();
 
-   setChart(chart: ChartClass) {
+  override setChart(chart: ChartClass) {
     this.chart = chart;
     google.visualization.events.addListener(
         this.chart, 'sort', (event: SortEvent) => {
@@ -52,7 +52,7 @@ export class StatsTableDataProvider extends DefaultDataProvider {
         });
   }
 
-   parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
+  override parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
     if (!data) return;
     const dataTable = new google.visualization.DataTable(data);
     if (this.hasDiff && this.diffTable) {
@@ -68,14 +68,14 @@ export class StatsTableDataProvider extends DefaultDataProvider {
         undefined;
   }
 
-   setFilters(filters: google.visualization.DataTableCellFilter[]) {
+  override setFilters(filters: google.visualization.DataTableCellFilter[]) {
     this.filters = filters;
     if (this.dataTable) {
       this.update.emit();
     }
   }
 
-   process(): google.visualization.DataTable|google.visualization.DataView|null {
+  override process(): google.visualization.DataTable|google.visualization.DataView|null {
     if (!this.dataTable) {
       return null;
     }
@@ -101,7 +101,7 @@ export class StatsTableDataProvider extends DefaultDataProvider {
     return dataView;
   }
 
-   getOptions(): ChartOptions|null {
+  override getOptions(): ChartOptions|null {
     this.options.sortAscending = this.sortAscending;
     this.options.sortColumn = this.sortColumn;
     return this.options;

diff --git a/frontend/app/components/graph_viewer/BUILD b/frontend/app/components/graph_viewer/BUILD
@@ -36,6 +36,7 @@ xprof_ng_module(
 sass_binary(
     name = "graph_viewer_css",
     src = "graph_viewer.scss",
+    # stack = True,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/graph_viewer/graph_config/BUILD b/frontend/app/components/graph_viewer/graph_config/BUILD
@@ -34,6 +34,7 @@ xprof_ng_module(
 sass_binary(
     name = "graph_config_css",
     src = "graph_config.scss",
+    # stack = True,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/hlo_stats/BUILD b/frontend/app/components/hlo_stats/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":hlo_stats_css",
         "hlo_stats.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/core",
         "@npm//@angular/forms",
@@ -44,6 +45,7 @@ xprof_ng_module(
 sass_binary(
     name = "hlo_stats_css",
     src = "hlo_stats.scss",
+    # stack = True,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/hlo_stats/hlo_stats.ts b/frontend/app/components/hlo_stats/hlo_stats.ts
@@ -192,7 +192,7 @@ export class HloStats extends Dashboard implements OnDestroy {
     };
   }
 
-   updateView() {
+  override updateView() {
     this.dataInfoForTable = {
       ...this.dataInfoForTable,
       filters: this.getFilters(),
@@ -247,7 +247,7 @@ export class HloStats extends Dashboard implements OnDestroy {
     this.dataInfoForTable.dataProvider.notifyCharts();
   }
 
-   parseData(data: SimpleDataTable | null) {
+  override parseData(data: SimpleDataTable | null) {
     if (!data) return;
     // Five charts share one DataProvider. In order to prevent DataTable from
     // being created multiple times, it calls DataProvider function directly.

diff --git a/frontend/app/components/inference_profile/BUILD b/frontend/app/components/inference_profile/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":inference_profile_css",
         "inference_profile.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/common",
         "@npm//@angular/core",

diff --git a/frontend/app/components/input_pipeline/BUILD b/frontend/app/components/input_pipeline/BUILD
@@ -33,6 +33,7 @@ xprof_ng_module(
 sass_binary(
     name = "input_pipeline_css",
     src = "input_pipeline.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/input_pipeline/analysis_summary/BUILD b/frontend/app/components/input_pipeline/analysis_summary/BUILD
@@ -22,6 +22,7 @@ xprof_ng_module(
 sass_binary(
     name = "analysis_summary_css",
     src = "analysis_summary.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/input_pipeline/device_side_analysis_detail/BUILD b/frontend/app/components/input_pipeline/device_side_analysis_detail/BUILD
@@ -14,6 +14,7 @@ xprof_ng_module(
         ":device_side_analysis_detail_css",
         "device_side_analysis_detail.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         ":device_side_analysis_detail_data_provider",
         "@npm//@angular/core",
@@ -37,6 +38,7 @@ ts_library(
 sass_binary(
     name = "device_side_analysis_detail_css",
     src = "device_side_analysis_detail.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/...s/input_pipeline/device_side_analysis_detail/device_side_analysis_detail_data_provider.ts b/...s/input_pipeline/device_side_analysis_detail/device_side_analysis_detail_data_provider.ts
@@ -9,7 +9,7 @@ export class DeviceSideAnalysisDetailDataProvider extends DefaultDataProvider {
     this.columnIds = columnIds;
   }
 
-   parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
+  override parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
     if (!data) return;
     const dataTable = new google.visualization.DataTable(data);
 

diff --git a/frontend/app/components/input_pipeline/host_side_analysis_detail/BUILD b/frontend/app/components/input_pipeline/host_side_analysis_detail/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":host_side_analysis_detail_css",
         "host_side_analysis_detail.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         ":host_side_analysis_detail_table_data_provider",
         "@npm//@angular/core",
@@ -40,6 +41,7 @@ xprof_ng_module(
 sass_binary(
     name = "host_side_analysis_detail_css",
     src = "host_side_analysis_detail.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/...input_pipeline/host_side_analysis_detail/host_side_analysis_detail_table_data_provider.ts b/...input_pipeline/host_side_analysis_detail/host_side_analysis_detail_table_data_provider.ts
@@ -8,7 +8,7 @@ export class HostSideAnalysisDetailTableDataProvider extends
     DefaultDataProvider {
   private readonly hasHostOpsChanged = new EventEmitter<boolean>();
 
-   parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
+  override parseData(data: SimpleDataTable|Array<Array<(string | number)>>|null) {
     if (!data) return;
     const dataTable = new google.visualization.DataTable(data);
     if (dataTable.getNumberOfRows() < 1) {

diff --git a/frontend/app/components/kernel_stats/BUILD b/frontend/app/components/kernel_stats/BUILD
@@ -44,6 +44,7 @@ xprof_ng_module(
 sass_binary(
     name = "kernel_stats_css",
     src = "kernel_stats.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",

diff --git a/frontend/app/components/kernel_stats/kernel_stats_table/BUILD b/frontend/app/components/kernel_stats/kernel_stats_table/BUILD
@@ -13,6 +13,7 @@ xprof_ng_module(
         ":kernel_stats_table_css",
         "kernel_stats_table.ng.html",
     ],
+    # strict_templates = False,
     deps = [
         "@npm//@angular/core",
         "@npm//@types/google.visualization",
@@ -26,6 +27,7 @@ xprof_ng_module(
 sass_binary(
     name = "kernel_stats_table_css",
     src = "kernel_stats_table.scss",
+    # stack = False,
     sourcemap = False,
     deps = [
         "@org_xprof//frontend/app/styles:common",