Description
The old inference_engine.py (GroundingDINO) exists alongside inference_engine_yolox.py. Docs say YOLOX is 47x faster but old code remains, confusing contributors.
Impact
- Low severity
- Code confusion
- Maintenance burden
Recommendation
Archive it - Keep for historical reference but clear it's not active.
Implementation
- Create
archive/grounding_dino/ directory
- Move old files there
- Add README explaining deprecation
- Update all imports
- Update documentation
Also Clean Up
- Config text_prompts (lines 137-231) for GroundingDINO
- Old model references
- Unused imports
Location
src/inference_engine.py
config/config.yaml:137-231
Description
The old
inference_engine.py(GroundingDINO) exists alongsideinference_engine_yolox.py. Docs say YOLOX is 47x faster but old code remains, confusing contributors.Impact
Recommendation
Archive it - Keep for historical reference but clear it's not active.
Implementation
archive/grounding_dino/directoryAlso Clean Up
Location
src/inference_engine.pyconfig/config.yaml:137-231