这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on Feb 13, 2025. It is now read-only.

Conversation

@madelynnblue
Copy link
Contributor

No description provided.

@madelynnblue
Copy link
Contributor Author

Other functions (des, dropge, drople, limit, nv) can also benefit from scalar promotion. But since they are not reduction functions we'll need another way to do the tag matching. They may come in a later PR. At least we get forecastlr and percentile now.

@kylebrandt
Copy link
Member

drop__ could add a fair amount of power I think, something like:

$series = q("...")
$limit = last(q("..."))
$filteredSeries = dropge($series, percentille($series, .95))
$daysLeft = forecastlr($filteredSeries, $limit) / d("1d")  

Unless there is another way to do this...

@madelynnblue
Copy link
Contributor Author

Yes, they will happen. There's just no obvious common way to do them all now, since they don't have the same function signature.

@madelynnblue
Copy link
Contributor Author

Ok, now does the drop functions. My previous note is somewhat wrong. NV and limit should always remain scalars. Des could use this, but in reality that doesn't really make sense.

@kylebrandt
Copy link
Member

docs need to be updated to reflect the drop funcs

@madelynnblue madelynnblue force-pushed the scalar-promotion branch 2 times, most recently from 6281f33 to 8585010 Compare July 7, 2015 17:01
@madelynnblue
Copy link
Contributor Author

Updated with dropg, dropl, and docs.

@kylebrandt
Copy link
Member

Will fix #1064

Copy link
Contributor

Choose a reason for hiding this comment

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

this is a really confusing data structure to only hold one test. Either add more or inline the fields (or make an anonymous struct for test cases).

@madelynnblue
Copy link
Contributor Author

Updated with better tests - now combined with the existing expr tests, it just knows about types now.

captncraig added a commit that referenced this pull request Jul 20, 2015
cmd/bosun: promote scalars to numberSets
@captncraig captncraig merged commit 3682907 into master Jul 20, 2015
@captncraig captncraig deleted the scalar-promotion branch July 20, 2015 18:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants