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

Add max keyword argument to difference/derivative functions. #722

Closed
adamperlin opened this issue Jul 3, 2018 · 3 comments
Closed

Add max keyword argument to difference/derivative functions. #722

adamperlin opened this issue Jul 3, 2018 · 3 comments
Assignees
@nathanielc
Labels
community enhancement New feature or request func/derivative func/difference no-issue-activity team/query

Comments

@adamperlin
Copy link
Contributor

From ifql created by nathanielc : influxdata/ifql#223

influxdata/influxdb#5954

The max helps account for integer wrap.
@mark-rushakoff mark-rushakoff transferred this issue from another repository Jan 8, 2019
@jsternberg
Copy link
Contributor

The function signature has to be discussed before working on this so if you are a community member planning to work on this, please consult us about API design before beginning your work. Thanks.

@davidby-influx
Copy link

@timhallinflux - this resurfaced in another issue in which I will tag you. I have provided a work-around in that issue, but this is obviously a better (but more expensive) solution for Flux to implement.

@github-actions GitHub Actions
Copy link

This issue has had no recent activity and will be closed soon.

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

@nathanielc nathanielc

Labels
community enhancement New feature or request func/derivative func/difference no-issue-activity team/query
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jsternberg @nathanielc @russorat @adamperlin @davidby-influx