Use has_bad_values() for restart check (catches NaN + Inf)

Replace has_NaNs() with the existing has_bad_values() method which
checks for both NaN and Inf, matching the straight-run check.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: sbryngelson

toolchain/mfc/packer/pack.py

@@ -40,12 +40,6 @@ def find(self, filepath: str) -> PackEntry:
     def set(self, entry: PackEntry):
         self.entries[entry.filepath] = entry
 
-    def has_NaNs(self) -> bool:
-        """Return True if any entry contains NaN values."""
-        import math
-
-        return any(math.isnan(d) for e in self.entries.values() for d in e.doubles)
-
     def remove(self, filepath_or_entry: typing.Union[str, PackEntry]):
         if isinstance(filepath_or_entry, str):
             del self.entries[filepath_or_entry]

toolchain/mfc/test/test.py

@@ -445,8 +445,8 @@ def _handle_case(case: TestCase, devices: typing.Set[int]):
             if restart_err is not None:
                 raise MFCException(f"Test {case}: Restart pack error: {restart_err}")
 
-            if restart_pack.has_NaNs():
-                raise MFCException(f"Test {case}: NaNs detected in restarted output.")
+            if restart_pack.has_bad_values():
+                raise MFCException(f"Test {case}: NaN or Inf detected in restarted output.")
 
             _, restart_msg = packtol.compare(restart_pack, straight_pack, packtol.Tolerance(tol, tol))
             if restart_msg is not None: