shiavm006 opened a new pull request, #606: URL: https://github.com/apache/mahout/pull/606
## Purpose of PR Fix a critical bug where `bind_parameters()` silently ignores unknown parameter names, causing parameters to remain unbound (None) and leading to cryptic errors during circuit execution. This fix adds immediate validation with clear error messages when users pass incorrect parameter names (typos, wrong names, etc.). ## Related Issues or PRs Fixes #603 ## Changes Made - [x] Bug fix - [ ] New feature - [ ] Refactoring - [ ] Documentation - [ ] Test - [ ] CI/CD pipeline - [ ] Other **Details:** - Added validation to check if parameter exists before binding - Raises `ValueError` with helpful message listing available parameters - Prevents silent failures and improves debugging experience - Follows same pattern as Qiskit's parameter validation ## Breaking Changes - [ ] Yes - [x] No **Note:** This is technically a breaking change for code that was passing invalid parameters, but such code was already broken (just failing silently). The fix makes the failure explicit and immediate. ## Checklist - [x] Added or updated unit tests for all changes - [x] Added or updated documentation for all changes - [x] Successfully built and ran all unit tests or manual tests locally - [x] PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue) - [x] Code follows ASF guidelines -- 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]
