[GitHub] incubator-quickstep issue #309: Added ProbabilityStore class

2017-10-12 Thread zuyu 
Github user zuyu commented on the issue:

https://github.com/apache/incubator-quickstep/pull/309
  
LGTM. Merging.


---

[GitHub] incubator-quickstep issue #309: Added ProbabilityStore class

2017-10-10 Thread hbdeshmukh 
Github user hbdeshmukh commented on the issue:

https://github.com/apache/incubator-quickstep/pull/309
  
Thanks for the comments. I have addressed all of them. The precision with 
floats is good enough for our use case. 


---