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

Implement sum of squared deviations for exponential histograms in SDK exporter #807

Open
dashpole opened this issue Mar 4, 2024 · 1 comment
Open

Implement sum of squared deviations for exponential histograms in SDK exporter #807

dashpole opened this issue Mar 4, 2024 · 1 comment
Labels
enhancement New feature or request priority: p3

Comments

@dashpole
Copy link
Contributor

dashpole commented Mar 4, 2024 •
edited

Follow-up to #777

See

opentelemetry-operations-go/exporter/metric/metric.go

Line 630 in c7b17b6

// TODO: Implement "setSumOfSquaredDeviationExpHist" for parameter "enableSOSD" functionality.

For fixed-bucket histograms, we assumed each observation occurred at the center of each bucket, and all underflow and overflow observations occurred at the lowest/highest bucket boundary. We could probably implement similar logic for exponential histograms.
@dashpole dashpole added enhancement New feature or request priority: p3 labels Mar 4, 2024
@dashpole
Copy link
Contributor Author

dashpole commented Mar 4, 2024

cc @franciscovalentecastro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request priority: p3
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dashpole