Increase precision of utils in op profile from 1 decimal point to 2

PiperOrigin-RevId: 624338266
tensorflow · Apr 13, 2024 · fd8bd17 · fd8bd17
1 parent 55ed9c7
commit fd8bd17
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 3 deletions.
diff --git a/frontend/app/common/utils/utils.ts b/frontend/app/common/utils/utils.ts
@@ -273,8 +273,8 @@ export function percent(fraction: number, defaultValueIfNull = '-'): string {
   if (fraction < 0.00001) {
     return '0.0%';
   } else {
-    // Round to 1 decimal place.
-    return `${(fraction * 100).toFixed(1)}%`;
+    // Round to 2 decimal place.
+    return `${(fraction * 100).toFixed(2)}%`;
   }
 }
 

diff --git a/frontend/app/components/op_profile/op_profile_common.scss b/frontend/app/components/op_profile/op_profile_common.scss
@@ -53,6 +53,19 @@
   cursor: pointer;
 }
 
+// set left margin of 16px to match leading span for each cell (for aria-label)
+.time-header,
+.wasted-header,
+.hbmFraction-header,
+.name-header,
+.provenance-header,
+.utilization-hader,
+.hbmUtilization-header,
+.utilization-header,
+.hbmUtilization-header {
+  margin-left: 16px;
+}
+
 .time-header,
 .wasted-header,
 .hbmFraction-header,
@@ -87,6 +100,11 @@
   font-size: smaller;
 }
 
+.wasted,
+.wasted-header {
+  width: 8%;
+}
+
 .name-header,
 .name {
   display: inline-block;
@@ -96,7 +114,8 @@
   word-break: break-all;
 }
 
-.name {
+.name,
+.provenance {
   text-align: left;
 }