[tmva][sofie] Remove Keras and PyTorch parses from ROOT repository

README/ReleaseNotes/v642/index.md

@@ -54,6 +54,9 @@ The following people have contributed to this new version:
 * The inclusion by external projects of Makefile templates contained within ROOT is deprecated in 6.42, a warning will be raised if you use them. These files will be removed in ROOT 7.
 * The conversion from Python set to **RooArgSet** is deprecated and won't work anymore in ROOT 6.44. The problem is that Python sets are unordered while RooArgSets are ordered, and this mismatch can lead to subtle problems later on. Prefer conversion from Python lists or tuples, which are ordered too.
 * The ROOT IO capability for the `TMVA::Experimental::SOFIE::RModel` has been removed. Users should not be encouraged to serialize models in experimental classes. For the serialization of ONNX models one can already use ONNX directly, and even serialize the ONNX bytes to a ROOT file if required.
+* The Keras and PyTorch parsers for SOFIE (`TMVA::Experimental::SOFIE::PyKeras` and `PyTorch`) are now removed, so `RSofieReader` only accepts ONNX files.
+These parsers relied on private implementation details of Keras and PyTorch, which change faster than is appropriate for ROOT's stability standards.
+Users are encouraged to export their models to ONNX and use the retained ONNX parser instead.
 * The ROOT IO capability for the `TMVA::Experimental::RBDT` class has been removed, along with the `TMVA.Experimental.SaveXGBoost` Python function. Experimental classes should not be persistified since their on-disk layout is not guaranteed to be stable. An `RBDT` is now built directly from an XGBoost model in its native JSON serialization with the new `TMVA::Experimental::RBDT::LoadXGBoost(jsonPath)`, which works both from C++ and Python. To convert a trained model, save it first with XGBoost's `Booster.save_model("model.json")` and then load it with `LoadXGBoost`.
 * The **JsMVA** feature for interactive TMVA training in Jupyter notebooks is now removed. It was not functional for years and was therefore already excluded from ROOT 6.38. This also removes the `TMVA::IPythonInteractive` class and the related interactive-training interfaces from the TMVA method and fitter classes, such as `MethodBase::ExitFromTraining()` or `FitterBase::SetIPythonInteractive()`.
 * The **RooStats::DebuggingSampler** and **RooStats::DebuggingTestStat** classes are removed. They were mock implementations of the `TestStatSampler` and `TestStatistic` interfaces that returned uniform random numbers independent of the data, only meant for debugging the RooStats framework itself during its initial development.

bindings/pyroot/pythonizations/python/CMakeLists.txt

@@ -59,32 +59,7 @@ if(tmva)
         ROOT/_pythonization/_tmva/_rbdt.py
         ROOT/_pythonization/_tmva/_rtensor.py
         ROOT/_pythonization/_tmva/_utils.py
-        ROOT/_pythonization/_tmva/_gnn.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/__init__.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/parser.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/__init__.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/batchnorm.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/binary.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/concat.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/conv.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/dense.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/elu.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/flatten.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/identity.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/layernorm.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/leaky_relu.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/permute.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/pooling.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/reshape.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/relu.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/rnn.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/selu.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/sigmoid.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/softmax.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/swish.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_keras/layers/tanh.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_pytorch/__init__.py
-        ROOT/_pythonization/_tmva/_sofie/_parser/_pytorch/parser.py)
+        ROOT/_pythonization/_tmva/_gnn.py)
 endif()
 
 set(py_sources

bindings/pyroot/pythonizations/python/ROOT/_facade.py

@@ -565,11 +565,6 @@ def TMVA(self):
             # The comment suppresses linter errors about unused imports.
             from ._pythonization import _tmva  # noqa: F401
             from ._pythonization._tmva._rtensor import _AsRTensor
-            from ._pythonization._tmva._sofie._parser._keras.parser import PyKeras
-            from ._pythonization._tmva._sofie._parser._pytorch.parser import PyTorch
-
-            setattr(ns.Experimental.SOFIE, "PyKeras", PyKeras)
-            setattr(ns.Experimental.SOFIE, "PyTorch", PyTorch)
 
             ns.Experimental.AsRTensor = _AsRTensor
         except ImportError: