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.

Sign up for GitHub

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

Jump to bottom

Merge upstream changes #119

Closed
wants to merge 21 commits into from
Closed

Merge upstream changes #119

wants to merge 21 commits into from

Conversation

carrieedwards
Copy link
Collaborator

This PR merges changes from upstream into this fork of CarbonAPI.

npazosmendez and others added 21 commits September 16, 2022 14:10
@npazosmendez
Fix exponentialMovingAverage panic
6d20987 
The panic was happening when the window size was greater than
the length of the values. Now if that happens no panics occur and
the average is returned. This matches graphiteweb.
@Civil
Merge pull request #698 from grafana/njpm/fix-ema-panic
cb942f5 
Fix exponentialMovingAverage panic
@carrieedwards
Add support for Graphite web toLowerCase function
b787997
@carrieedwards
Add tests for toLowerCase function
27fbcdb
@carrieedwards
Update glue.go
405f2d7
@carrieedwards
Update COMPATIBILITY.md to show support of toLowerCase
978fed2
@carrieedwards
Add support for Graphite web toUpperCase function
4510493
@carrieedwards
Add tests for toUpperCase function
303d8d5
@carrieedwards
Update glue.go
1596d29
@carrieedwards
Update COMPATIBILITY.md to include toUpperCase function
5c98794
@carrieedwards
Add support for Graphite web sumSeriesLists function
e7f0f67
@carrieedwards
Add tests for sumSeriesLists
1626ef2
@carrieedwards
Update glue.go
28e2e5b
@carrieedwards
Update COMPATIBILITY.md to include sumSeriesLists
0107dde
@Civil
Merge pull request #699 from go-graphite/cedwards/add-toLowerCase-fun…
e8ebd53 
…ction

Add support for Graphite web toLowerCase function
@Civil
Merge branch 'main' into cedwards/add-toUpperCase-function
c7147c0
@Civil
Merge pull request #700 from go-graphite/cedwards/add-toUpperCase-fun…
ec9a669 
…ction

Add support to Graphite web toUpperCase function
@Civil
Fix problems that I accidentally introduced while trying to resolve m…
53ec5fe 
…erge conflict
@Civil
Another attempt to fix previous problem
220b60d 
 Apparantly there is a directory that gen.go wasn't expecting and
 `go generate` in expr/functions produced `glue.go` file that didn't compile

 fix both generator and glue
@carrieedwards
Merge branch 'main' into cedwards/add-sumSeriesLists-function
547774c
@Civil
Merge pull request #701 from go-graphite/cedwards/add-sumSeriesLists-…
1149075 
…function

Add support for Graphite web sumSeriesLists function
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ npazosmendez
✅ carrieedwards
❌ Civil
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@npazosmendez npazosmendez npazosmendez approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@carrieedwards @CLAassistant @npazosmendez @Civil