review comments

Author: FFroehlich

pytest.ini

@@ -13,7 +13,6 @@ filterwarnings =
     ignore:Conservation laws for non-constant species in models with RateRules are currently not supported and will be turned off.:UserWarning
     ignore:Conservation laws for non-constant species in models with Species-AssignmentRules are currently not supported and will be turned off.:UserWarning
     ignore:Conservation laws for non-constant species in combination with parameterized stoichiometric coefficients are not currently supported and will be turned off.:UserWarning
-    ignore:PEtab v2.0.0 mapping tables are only partially supported:UserWarning
     ignore:Support for PEtab2.0.*experimental:UserWarning
     ignore:The JAX module is experimental and the API may change in the future.:ImportWarning
     # hundreds of SBML <=5.17 warnings

python/sdist/amici/sbml_import.py

@@ -215,7 +215,7 @@ def _process_document(self) -> None:
         log_execution_time("validating SBML", logger)(
             self.sbml_doc.validateSBML
         )()
-        # _check_lib_sbml_errors(self.sbml_doc, self.show_sbml_warnings)
+        _check_lib_sbml_errors(self.sbml_doc, self.show_sbml_warnings)
 
         # Flatten "comp" model? Do that before any other converters are run
         if any(
@@ -262,7 +262,7 @@ def _process_document(self) -> None:
         # If any of the above calls produces an error, this will be added to
         # the SBMLError log in the sbml document. Thus, it is sufficient to
         # check the error log just once after all conversion/validation calls.
-        # _check_lib_sbml_errors(self.sbml_doc, self.show_sbml_warnings)
+        _check_lib_sbml_errors(self.sbml_doc, self.show_sbml_warnings)
 
         # need to reload the converted model
         self.sbml = self.sbml_doc.getModel()