mahout icon indicating copy to clipboard operation
mahout copied to clipboard

[QDP] fix test bug

Open ryankert01 opened this issue 3 weeks ago • 6 comments

Purpose of PR

default has been changed to f32, so the bits result would be 32*2=64 solution: Make the test not init the executor to default, instead init to a predetermined f64

Related Issues or PRs

Closes #733

Changes Made

  • [ ] Bug fix
  • [ ] New feature
  • [ ] Refactoring
  • [ ] Documentation
  • [ ] Test
  • [ ] CI/CD pipeline
  • [ ] Other

Breaking Changes

  • [ ] Yes
  • [ ] No

Checklist

  • [ ] Added or updated unit tests for all changes
  • [ ] Added or updated documentation for all changes
  • [ ] Successfully built and ran all unit tests or manual tests locally
  • [ ] PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue)
  • [ ] Code follows ASF guidelines

TODO

  • [ ] add a test for original new() fp32, bring back the coverage

ryankert01 avatar Dec 17 '25 04:12 ryankert01