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

RFC: add incrtabulate #88

Open
4 tasks done
kgryte opened this issue Apr 22, 2017 · 0 comments
Open
4 tasks done

RFC: add incrtabulate #88

kgryte opened this issue Apr 22, 2017 · 0 comments
Labels
difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. Enhancement Issue or pull request for enhancing existing functionality. Feature Issue or pull request for adding a new feature. Math Issue or pull request specific to math functionality. Needs Discussion Needs further discussion. RFC Request for comments. Feature requests and proposed changes.

Comments

@kgryte
Copy link
Member

kgryte commented Apr 22, 2017

Checklist

Please ensure the following tasks are completed before filing an issue.

  • Read and understood the Code of Conduct.
  • Searched for existing issues and pull requests.
  • If this is a general question, searched the FAQ for an existing answer.
  • If this is a feature request, the issue name begins with RFC: .

Description

Description of the issue (or feature request).

This RFC proposes to add @stdlib/math/statistics/incr/tabulate. The package would incrementally generate a frequency table.

Related Issues

Does this issue (or feature request) have any related issues?

No.

Questions

Any questions for reviewers?

  • Open question whether this package should accept an "indicator" function which can process an input value before tabulation, similar to @stdlib/utils/tabulate-by.
  • Should there exist a separate package for @stdlib/math/statistics/incr/tabulate-by which allows pre-processing, as described above?

Other

Any other information relevant to this issue (or feature request)? This may include screenshots, references, stack traces, sample output, and/or implementation notes.

Based on the @stdlib/utils/tabulate algorithm, need to provide a warning to be careful with "diverse" input streams, as performance decreases with number of unique values. Hence, this function would not be appropriate for tabulating real numbers without first processing input to limit the possible range of input values.

@kgryte kgryte added difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. Enhancement Issue or pull request for enhancing existing functionality. Feature Issue or pull request for adding a new feature. Math Issue or pull request specific to math functionality. Needs Discussion Needs further discussion. RFC Request for comments. Feature requests and proposed changes. labels Apr 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: 1 Low degree of difficulty. Should be straightforward to implement and/or resolve. Enhancement Issue or pull request for enhancing existing functionality. Feature Issue or pull request for adding a new feature. Math Issue or pull request specific to math functionality. Needs Discussion Needs further discussion. RFC Request for comments. Feature requests and proposed changes.
Projects
None yet
Development

No branches or pull requests

1 participant