More appropriate way of doing the same thing

q2_feature_table/_summarize/summarize_assets/sample-frequency-detail.html

@@ -191,8 +191,12 @@ <h3>Plot Controls</h3>
 
     // start the counter at 1 to ignore the header row
     for (var i = 1; row = table.rows[i]; i++) {
-      let sampleFrequency = row.cells[1].innerText;
-      sampleFrequency = strToInt(sampleFrequency);
+      // The value in table is formatted as a string for display to the user,
+      // but the value in sampleFrequencies is formatted as an int for data
+      // manipulation, so we use the ID we are on in table to get the frequency
+      // fro sampleFrequencies.
+      const sampleID = sampleFrequencies[row.cells[0].innerText];
+      const sampleFrequency = sampleFrequencies[sampleID];
 
       if (sampleFrequency < samplingDepth) {
         row.className = "danger";

q2_feature_table/_summarize/summarize_assets/utils/util.js

@@ -31,9 +31,3 @@ function sortIDs(a, b, data) {
 
     return diff;
 }
-
-
-// Parse a string that may contain commas into a base 10 integer
-function strToInt(val) {
-    return parseInt(val.replaceAll(",", ""), 10);
-}