yehfelareborn opened a new pull request, #1179:
URL: https://github.com/apache/mahout/pull/1179

   ### Related Issues
   
   <!-- Closes #1178  -->
   
   ### Changes
   
   - [ ] Bug fix
   - [ ] New feature
   - [ ] Refactoring
   - [ ] Documentation
   - [ ] Test
   - [ ] CI/CD pipeline
   - [ ] Other
   
   ### Why
   
   - angle and basis encoding were already implemented in Rust (angle.rs, 
basis.rs), but had no Python test coverage
   - The existing test_encode_numpy_encoding_methods only covered amplitude and 
left a TODO comment for the missing methods
   
   ### How
   
   Add angle encoding tests: 1D single sample, 2D multi-sample, loading from 
.npy file, and a negative test for wrong input size
   Add basis encoding tests: 1D single sample, 2D multi-sample, loading from 
.npy file
   Add basis state correctness test: verify the encoded output is the correct 
one-hot vector (|basis_index⟩)
   Add basis negative tests: negative index, non-integer index, out-of-range 
index
   Remove the TODO comment
   
   ## Checklist
   
   - [ ] Added or updated unit tests for all changes
   - [ ] Added or updated documentation for all changes
   


-- 
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]

Reply via email to