Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memoize Histogram Count #23

Open
vjkoskela opened this issue May 2, 2020 · 0 comments
Open

Memoize Histogram Count #23

vjkoskela opened this issue May 2, 2020 · 0 comments

Comments

@vjkoskela
Copy link
Contributor

Since we now compute the count of samples in a histogram on construction:

In order to support originalCount we should just memoize the count for purposes of sample counting as well. Of course, we'll need to handle the other case where we have a filtered histogram where the sample count differs from the original count. However, that is not the common case in my opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant