Skip to content

Evaluation on Windows fails due to missing ninja (use standard COCOeval) #1837

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
cmaranes opened this issue May 10, 2025 · 0 comments
Open

Comments

@cmaranes
Copy link

On Windows, the evaluation step tries to import the optimized Cython version (COCOeval_opt), which in turn requires a working ninja install for JIT compilation. If ninja isn’t available, the import fails and the entire eval pipeline breaks.

🔧 Temporary Workaround

Force use of the pure-Python evaluator by swapping out the import. In
yolox/evaluators/coco_evaluator.py (around line 294):

from yolox.layers import COCOeval_opt as COCOeval

, replace:

- from yolox.evaluators.coco_opt import COCOeval_opt
+ from pycocotools.cocoeval import COCOeval

This change bypasses the ninja dependency and lets evaluation complete using the standard COCO API.

Feel free to let me know if you’d like a PR implementing the fallback logic!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant