[mlpack-git] [mlpack/mlpack] Heaps for mlpack! (#732)

[Ryan Curtin]

I thought that `std::pair<>` would be better to use here since it is a more well-known class, and allows us to avoid declaring the `Candidate` class many different places in the code.  Do you think there is a compelling reason to keep the `Candidate` class?  If the thought is that the `Candidate` name makes it clear, we can just add a comment like this:

```
// Holds an (index, score) pair.
std::pair<size_t, double> candidate;
```

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/732#issuecomment-234966216
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160725/ea609a70/attachment.html>