search

iu-parfunc / lvars

The LVish Haskell library
http://hackage.haskell.org/package/lvish
80 stars 14 forks source link

Get mergesort running with the test-suite #113

Closed rrnewton closed 9 years ago

rrnewton commented 9 years ago

After everything is green on our CI unit testing, mergesort should really be included as a test.

Currently it has its own cabal file. But now that cabal has "benchmark" sections, it could be included as a benchmark of the top-level par-transformers cabal file.

osa1 commented 9 years ago

I just implemented QuickCheck tests for this: https://github.com/iu-parfunc/lvars/commit/77a6a36fc38e733656c5a20345adaf4c3173465e#diff-1e8a7ec63946a27fbec1217374f65e69R26 unfortunately it's failing all the time. Looks like I have some debugging to do...

osa1 commented 9 years ago

I fixed couple of bugs and implemented some tests(including QuickCheck test for sorting random vectors). Now only problem is making it running in travis.

osa1 commented 9 years ago

par-mergesort is now ready but see #114.

osa1 commented 9 years ago

This is done, Travis is green now.