Add Numeric class test - Githubissues

lehins / massiv

Efficient Haskell Arrays featuring Parallel computation

BSD 3-Clause "New" or "Revised" License

386 stars 23 forks source link

Add Numeric class test #101

Closed TristanCacqueray closed 3 years ago

TristanCacqueray commented 4 years ago

[x] Bump up the version in cabal file
[x] Any changes that could be relevant to users have been recorded in the CHANGELOG.md
[x] The documentation has been updated, if necessary.
[x] Property tests or at least some unit test cases been added for all new functionality.
[x] Link to any issues that might be related to this Pull Request.

TristanCacqueray commented 4 years ago

This meant to improve coverage but I failed at using the Arbitrary class for Numeric array. For some reason, ix or e seem always ambiguous. Without it, the extra spec test is rather limited.

lehins commented 4 years ago

This is a good start. I plan to do quite a bit of work on numeric operations at some point, I'll use this PR as a starting point for tests, but for now I'll just keep it opened in case you figure out how to add more tests.

lehins commented 3 years ago

Since newest version massiv-0.5.6 there now many tests for numeric functions with ~80% coverage. Closing