remove an old abort on jacobian for simplified-SDC

not only is it no longer true, it also did not work because
the extern runtime parameters were not initialized at this point

.clang-tidy

@@ -16,7 +16,12 @@ Checks: >
        -cppcoreguidelines-no-malloc,
        -cppcoreguidelines-non-private-member-variables-in-classes,
        -cppcoreguidelines-owning-memory,
-       -cppcoreguidelines-pro-*,
+       -cppcoreguidelines-pro-bounds-pointer-arithmetic,
+       -cppcoreguidelines-pro-type-cstyle-cast,
+       -cppcoreguidelines-pro-bounds-array-to-pointer-decay,
+       -cppcoreguidelines-pro-bounds-constant-array-index,
+      modernize-*,
+
      modernize-*,
        -modernize-avoid-c-arrays,
        -modernize-use-trailing-return-type,

Source/driver/Castro.cpp

@@ -322,13 +322,6 @@ Castro::read_params ()
     }
 #endif
 
-#ifdef REACTIONS
-#ifdef SIMPLIFIED_SDC
-    if (jacobian == 1) {
-      amrex::Abort("Simplified SDC requires the numerical Jacobian now (jacobian = 2)");
-    }
-#endif
-#endif
     // sanity checks
 
     if (grown_factor < 1) {