Merge pull request #203 from flatironinstitute/draws_table_wrapping

Improve formatting of draws table
flatironinstitute · Aug 6, 2024 · ba3756f · ba3756f
2 parents bd51afd + ec34271
commit ba3756f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/gui/src/app/SamplerOutputView/SamplerOutputView.tsx b/gui/src/app/SamplerOutputView/SamplerOutputView.tsx
@@ -108,7 +108,9 @@ const DrawsView: FunctionComponent<DrawsViewProps> = ({
   >(300);
   const formattedDraws = useMemo(() => {
     if (abbreviatedToNumRows === undefined) return draws;
-    return draws.map((draw) => draw.slice(0, abbreviatedToNumRows));
+    return draws.map((draw) =>
+      formatDraws(draw.slice(0, abbreviatedToNumRows)),
+    );
   }, [draws, abbreviatedToNumRows]);
   const handleExportToCsv = useCallback(() => {
     const csvText = prepareCsvText(
@@ -173,7 +175,7 @@ const DrawsView: FunctionComponent<DrawsViewProps> = ({
             <tr key={i}>
               <td>{drawChainIds[i]}</td>
               <td>{drawNumbers[i]}</td>
-              {draws.map((draw, j) => (
+              {draws2.map((draw, j) => (
                 <td key={j}>{draw[i]}</td>
               ))}
             </tr>
@@ -196,6 +198,11 @@ const DrawsView: FunctionComponent<DrawsViewProps> = ({
   );
 };
 
+const formatDraws = (draws: number[]) => {
+  if (draws.every((x) => Number.isInteger(x))) return draws;
+  return draws.map((x) => x.toPrecision(6));
+};
+
 const prepareCsvText = (
   draws: number[][],
   paramNames: string[],

diff --git a/gui/src/draws-table.css b/gui/src/draws-table.css
@@ -22,6 +22,7 @@
 .draws-table th,
 .draws-table td {
   padding: 2px 2px;
+  white-space: nowrap;
 }
 
 .draws-table tbody tr {