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

Handle case when first value is 0 in test_cumulative_aggregation_with_random_data #4139

Merged
merged 3 commits into from
Aug 21, 2024

Conversation

ocelotl
Copy link
Contributor

@ocelotl ocelotl commented Aug 20, 2024

This is a PR to show the failure mentioned in #4138.

Adding this change to show how the test case fails with a particular seed value.

@ocelotl ocelotl changed the title This test case fails with the change in this commit Handle case when first value is 0 in test_cumulative_aggregation_with_random_data Aug 20, 2024
@ocelotl ocelotl added Skip Changelog PRs that do not require a CHANGELOG.md entry and removed bug Something isn't working labels Aug 20, 2024
…onential_bucket_histogram_aggregation.py

Co-authored-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
@lzchen lzchen merged commit 6e1429e into open-telemetry:main Aug 21, 2024
376 checks passed
hyoinandout pushed a commit to hyoinandout/opentelemetry-python that referenced this pull request Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metrics Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants