Repository navigation
feat(device): add Apple Silicon MPS device support - #11
felipebridge merged 1 commit into
Conversation
|
Gracias por el PR. Reviso el diff completo (no solo la descripción) y esto es lo que pienso: Apruebo: DeviceType.MPS en config/settings.py Es un cambio mínimo, correcto, y bien testeado (los mocks de torch cubren cuda/mps/ninguno/sin-torch). Por favor saquen del PR: El cambio en scripts/check_environment.py que reemplaza except ConfigError por except Exception — eso silencia errores que no son de config, no tiene que ver con MPS y no está mencionado en la descripción. Si quieren mantener el mensaje de "Acceleration: MPS available", ok, pero sin tocar el manejo de excepciones. Con el PR reducido a solo DeviceType/resolve_device/tests lo mergeo sin problema. |
8dda0e1 to
8fe5dcf
Compare
|
Listo, totalmente de acuerdo con el alcance y el punto sobre CI. Se revirtieron los cambios en |
What does this change
mps) device acceleration support toDeviceTypeandresolve_device(), allowing macOS users on M-series chips to benefit from GPU-accelerated YOLO inference.tests/test_device.pyand configuration parsing test intests/test_config.py.Related issue
N/A
How was this tested
ruff check src tests dashboard scripts(Passed with 0 errors)tests/test_device.pyandtests/test_config.pycovering CPU, CUDA, MPS, auto fallback, and missing torch scenariosChecklist