search

PredictionIO / template-scala-parallel-complementarypurchase

PredictionIO Complementary Purchase Engine Template (Scala-based parallelized engine)
16 stars 22 forks source link

Add minimum basket size that is considered during recommendations #1

Closed pawel-madurski closed 9 years ago

pawel-madurski commented 9 years ago

Solution that provided Kenneth Chan on PIO user group works like a charm. Thanks

k4hoo commented 9 years ago

Great! Thanks @pawel-madurski for trying this new param!

Would you sign the contributor agreement? http://prediction.io/cla

Looks like the new param in missing in "case class AlgorithmParams" definition?

Would you add this new param to our doc (parameter description) as well? https://github.com/PredictionIO/PredictionIO/blob/fe1cc41af2d5aa8ce86e3c8ce881f6581f9db6e5/docs/manual/source/templates/complementarypurchase/dase.html.md.erb#L221

You can make PR to livedoc branch so the doc can go live immediately! :)

Thanks!

pawel-madurski commented 9 years ago

Kenneth. Sorry for delay but I had hard time. I've signed contributor agreement and edited doc. I'm not so good in all github stuff. I hope that you will be able to marge my pull request.

k4hoo commented 9 years ago

No problem! @pawel-madurski, Thanks a lot!