Skip to content

Commit

Permalink
Add comment about the checks not being comprehensive
Browse files Browse the repository at this point in the history
  • Loading branch information
jmcarcell committed Sep 5, 2024
1 parent c1da2ad commit 6afb58f
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions k4FWCore/python/k4FWCore/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down

0 comments on commit 6afb58f

Please sign in to comment.