rich7420 commented on PR #1311: URL: https://github.com/apache/mahout/pull/1311#issuecomment-4427202572
closing this one for now after re-evaluating the plan. the original motivation was a 2-D `(encoding, backend)` dispatch table to route between cuda and amd encoders in rust. but the amd path landed entirely in python (`qumat_qdp/backends/amd.py`), so there's no rust-side amd encoder for the enum to dispatch to. that makes the enum decorative — no caller actually needs it right now. revisiting if/when: - a rust-side amd impl is proposed, or - a future feasibility planner needs to introspect compute target as a typed value branch stays on origin so it's easy to revive. no rush. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
