Merge pull request #2 from ironprogrammer/dont-correct-data

Tested locally and LGTM.

Author: ironprogrammer

README.md

@@ -8,19 +8,17 @@ Please consider helping me cover my kid's swim meet fees!
 
 <a href="https://www.buymeacoffee.com/ironprogrammer" target="_blank"><img src="https://cdn.buymeacoffee.com/buttons/v2/default-yellow.png" alt="Buy Me A Coffee" style="height: 60px !important;width: 217px !important;" ></a>
 
-## Data Corrections - 2024-2025 Time Standards
+## Data Inconsistencies ⚠️
 
-The following guesstimated "corrections" were applied to clearly inconsistent data\* in the source PDF from the OSI site:
+The following issues were observed in the source data from the OSI site, and are highlighted in the app:
 
-| Age/Gender | Event | Course | Standard | PDF Value | JSON Value | Status | Reason |
-|------------|-------|--------|----------|-----------|------------|--------|--------|
-| Girls 9 | 100 Breast | LCM | A | `1:96.99` | `1:56.99` | ✅ Corrected | Invalid format |
-| Girls 10 | 100 Fly | LCM | B+ | `125:79.19` | `1:54.19` | ✅ Corrected | Possible OCR error |
-| Girls 13 | 200 Breast | SCM | B | `2:51.89`** | `2:51.89` | ⚠️ Error | B faster than A/B+ |
+| Age/Gender | Event      | Course | Standard | Value       | Issue              |
+| ---------- | ---------- | ------ | -------- | ----------- | ------------------- |
+| Girls 9    | 100 Breast | LCM    | A        | `1:96.99`   | Invalid format      |
+| Girls 10   | 100 Fly    | LCM    | B+       | `125:79.19` | Invalid format      |
+| Girls 13   | 200 Breast | SCM    | B        | `2:51.89`   | Invalid progression |
 
-\* *The OSI PDF may contain errors; and so can this app! Refer directly OSI for questions/corrections.*
-
-\** *Value not updated in JSON, however `3:31.89` might be the expected value based on time progression for this event.*
+\* *The OSI PDF may contain errors, and so can this app! Refer directly to OSI for questions/corrections.*
 
 > 🪲 Encounter any other issues? Head over to the [Issues tab](https://github.com/ironprogrammer/swimcheck/issues) and let me know!
 

index.html

@@ -100,6 +100,12 @@
             transition: background-color 0.3s ease;
         }
 
+        /* Row with invalid standard gets light yellow background on all cells */
+        .event-table tr.has-invalid-standard td {
+            background-color: #fff9e6;
+            transition: background-color 0.3s ease;
+        }
+
         /* Standard cells */
         .standard-cell {
             font-family: 'Courier New', monospace;
@@ -483,6 +489,12 @@ <h1>Swim Check 🏊</h1>
                 // Add 'has-value' class to row if user has entered a time
                 // Add met class to row for input border styling
                 let rowClass = userTime ? 'has-value' : '';
+
+                // Add 'has-invalid-standard' class if any standard has warning emoji
+                if (hasInvalidStandard(times)) {
+                    rowClass += ' has-invalid-standard';
+                }
+
                 if (qualification === 'a') {
                     rowClass += ' met-a';
                 } else if (qualification === 'b-plus') {
@@ -636,9 +648,10 @@ <h1>Swim Check 🏊</h1>
             const userSeconds = timeToSeconds(userTime);
             if (userSeconds === null) return null;
 
-            const aTime = standards.A ? timeToSeconds(standards.A) : null;
-            const bPlusTime = standards['B+'] ? timeToSeconds(standards['B+']) : null;
-            const bTime = standards.B ? timeToSeconds(standards.B) : null;
+            // Skip standards with warning emoji (invalid data)
+            const aTime = (standards.A && !standards.A.includes('⚠️')) ? timeToSeconds(standards.A) : null;
+            const bPlusTime = (standards['B+'] && !standards['B+'].includes('⚠️')) ? timeToSeconds(standards['B+']) : null;
+            const bTime = (standards.B && !standards.B.includes('⚠️')) ? timeToSeconds(standards.B) : null;
 
             // Lower times are better in swimming
             if (aTime !== null && userSeconds <= aTime) {
@@ -652,11 +665,19 @@ <h1>Swim Check 🏊</h1>
             }
         }
 
+        // Check if any standard has invalid data (contains warning emoji)
+        function hasInvalidStandard(times) {
+            return (times.A && times.A.includes('⚠️')) ||
+                   (times['B+'] && times['B+'].includes('⚠️')) ||
+                   (times.B && times.B.includes('⚠️'));
+        }
+
         // Convert time string to seconds
         function timeToSeconds(timeStr) {
             if (!timeStr) return null;
 
-            timeStr = timeStr.trim();
+            // Strip non-time characters (only allow numerics, ':', and '.')
+            timeStr = timeStr.trim().replace(/[^0-9:.]/g, '');
 
             // Format: MM:SS.MS or SS.MS
             const parts = timeStr.split(':');

swim_time_standards.json

@@ -205,7 +205,7 @@
               "name": "100 Breast",
               "SCY": { "A": "1:43.99", "B+": "1:49.19", "B": "1:54.39" },
               "SCM": { "A": "1:54.39", "B+": "2:00.19", "B": "2:05.89" },
-              "LCM": { "A": "1:56.99", "B+": "2:02.79", "B": "2:08.59" }
+              "LCM": { "A": "1:96.99⚠️", "B+": "2:02.79", "B": "2:08.59" }
             },
             {
               "name": "50 Fly",
@@ -362,7 +362,7 @@
               "name": "100 Fly",
               "SCY": { "A": "1:29.39", "B+": "1:33.89", "B": "1:38.39" },
               "SCM": { "A": "1:38.39", "B+": "1:43.39", "B": "1:48.29" },
-              "LCM": { "A": "1:48.49", "B+": "1:54.19", "B": "1:59.39" }
+              "LCM": { "A": "1:48.49", "B+": "125:79.19⚠️", "B": "1:59.39" }
             },
             {
               "name": "100 IM",
@@ -1160,7 +1160,7 @@
             {
               "name": "200 Breast",
               "SCY": { "A": "2:54.69", "B+": "3:03.39", "B": "3:12.59" },
-              "SCM": { "A": "3:12.19", "B+": "3:21.79", "B": "2:51.89" },
+              "SCM": { "A": "3:12.19", "B+": "3:21.79", "B": "2:51.89⚠️" },
               "LCM": { "A": "3:11.89", "B+": "3:21.39", "B": "3:31.49" }
             },
             {