Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Add maxSeries function #742

Merged
merged 1 commit into from
Oct 10, 2017
Merged

Conversation

shanson7
Copy link
Collaborator

@shanson7 shanson7 commented Oct 4, 2017

This code is mostly copy/pasta from sumSeries.

I jumped directly to stable since the code is rather simple. I spot-checked this against graphite and the only difference I noticed was some double-precision differences (e.g. graphite says 99.8005834878 and MT says 99.806, which is was MT returned in the raw render as well).

@shanson7
Copy link
Collaborator Author

shanson7 commented Oct 9, 2017

Any love?

@Dieterbe Dieterbe self-requested a review October 10, 2017 02:43
Copy link
Contributor

@Dieterbe Dieterbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. thanks @shanson7 !

@Dieterbe Dieterbe merged commit 15e56d4 into grafana:master Oct 10, 2017
@Aergonus Aergonus deleted the feature_MaxSeries branch January 8, 2018 20:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants