Florence2 nodes fail with 'attempted relative import with no known parent package' after TTS-Audio-Suite RVC runs (models name collision)

[zz13526585541-hash]

Environment

• ComfyUI 0.29.2, Python 3.12.10, torch 2.9.1+cu128, Windows
• ComfyUI-RMBG 3.1.0 + TTS-Audio-Suite (latest)

Steps to reproduce

1. Start ComfyUI, run any TTS-Audio-Suite voice-changer (RVC) workflow once. This inserts TTS-Audio-Suite\engines\rvc\impl\lib\infer_pack into sys.path[0] at runtime (see minimal_reference_wrapper.py).
2. Run a workflow using any Florence2 node (e.g. CogFlorence-2.2-Large). Model files download/load fine.

Expected: Florence2 inference runs.

Actual: Node fails with:

File "py\AILab_Florence2.py", line 168, in _get_model
    from models.modeling_florence2 import Florence2ForConditionalGeneration
  File "TTS-Audio-Suite\engines\rvc\impl\lib\infer_pack\models.py", line 5, in <module>
    from . import modules
ImportError: attempted relative import with no known parent package

Root cause
AILab_Florence2.py uses a bare top-level import from models.modeling_florence2 import .... Python's PathFinder resolves models by walking sys.path in order; after TTS-Audio-Suite's RVC engine inserts its infer_pack directory at sys.path[0], the plain file infer_pack\models.py shadows this repo's models/ package. That file is then imported as a top-level models module, and its internal from . import modules fails (no parent package). The error is intermittent: it only appears after an RVC workflow has run in the same process.

Suggested fix
Resolve the import without relying on a bare models name, e.g. insert this repo's own models dir at sys.path[0] before importing:

sys.path.insert(0, os.path.normpath(os.path.join(os.path.dirname(os.path.abspath(__file__)), "..", "models")))
from modeling_florence2 import Florence2ForConditionalGeneration

(This is what I applied locally and it works; modeling_florence2.py has no relative imports so it loads fine as a top-level module.)

Additionally, TTS-Audio-Suite should be asked to stop globally polluting the process namespace: pushing a directory containing a generic models.py onto sys.path[0] at runtime is a landmine for any other custom node doing bare-name imports.

[1038lab]

Thank you so much for the detailed breakdown and for tracing this all the way down to sys.path pollution! You nailed the root cause.

Dynamic sys.path.insert(0, ...) with generic script names (like a flat models.py) by third-party nodes is a known issue in ComfyUI's shared Python process environment. When another node places its directory at sys.path[0], any from models... import in subsequent nodes gets hijacked.

Here is how we will handle this:

• Defensive Fix in ComfyUI-RMBG: We will update AILab_Florence2.py to directly resolve and prioritize our repository's local models/ directory before importing modeling_florence2, ensuring external models.py files won't break the import chain.
• TTS-Audio-Suite Behavior: We strongly encourage opening an issue on the TTS-Audio-Suite repository as well. Custom nodes should avoid injecting directories containing generic file/folder names (models.py, utils.py, etc.) into sys.path[0] at runtime, as it pollutes the global process space for all nodes.

Thanks again for taking the time to report this and providing a clean workaround!