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

Probability sampler should ignore the parent sampled flag #150

Closed
jtescher opened this issue Jul 17, 2020 · 1 comment · Fixed by #154
Closed

Probability sampler should ignore the parent sampled flag #150

jtescher opened this issue Jul 17, 2020 · 1 comment · Fixed by #154
Labels
good first issue Good for newcomers

Comments

@jtescher
Copy link
Member

Probability sampler should always sample, and ignore any parent flags. See the spec change for details.

@jtescher jtescher added the good first issue Good for newcomers label Jul 17, 2020
@craigpastro
Copy link
Contributor

Do you mind if I give this one a shot?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants