From 6afb58f6d4dadd3aac36a1d398a5a1852171e0f7 Mon Sep 17 00:00:00 2001 From: jmcarcell Date: Thu, 5 Sep 2024 12:28:09 +0200 Subject: [PATCH] Add comment about the checks not being comprehensive --- k4FWCore/python/k4FWCore/utils.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/k4FWCore/python/k4FWCore/utils.py b/k4FWCore/python/k4FWCore/utils.py index 1aa31d9f..ec257139 100644 --- a/k4FWCore/python/k4FWCore/utils.py +++ b/k4FWCore/python/k4FWCore/utils.py @@ -30,6 +30,12 @@ def check_wrong_imports(code: str) -> None: from Configurables instead of k4FWCore. If such an import is found, an ImportError is raised. + If IOSvc and ApplicationMgr are not imported from k4FWCore it's possible + that k4run will fail, for example, by not adding the reader and writer + algorithms manually, which is done by the wrapper in k4FWCore. The checks + are not comprehensive, it's still possible to import IOSvc or ApplicationMgr + from Configurables. + Args: code (str): The code to check for wrong imports.