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

[C++] random::RandomArrayGenerator::Boolean "probability" misdocumented / incorrect #25251

Closed
asfimport opened this issue Jun 15, 2020 · 1 comment
Closed

[C++] random::RandomArrayGenerator::Boolean "probability" misdocumented / incorrect #25251

asfimport opened this issue Jun 15, 2020 · 1 comment
Assignees
@wesm
Labels
Component: C++ Type: bug
Milestone
1.0.0

Comments

@asfimport
Copy link
Collaborator

I would expect rng.Boolean(length, 0.99) to generate an array that is 99% "true" on average.

The docs say "/// \param[in] probability the estimated number of active bits", but this is not what the function does (the opposite, actually).

Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney / @wesm

Note: This issue was originally created as ARROW-9142. Please see the migration documentation for further details.
@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
I did this in d0f3b5f

@asfimport asfimport added this to the 1.0.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wesm wesm

Labels
Component: C++ Type: bug
Projects
None yet
Milestone
1.0.0
Development

No branches or pull requests
2 participants
@wesm @asfimport