[1617] Add ConstnativeHistogram #1654

shivanthzen · 2024-10-16T12:18:54Z

Fixes: #1617
Add exported ConstNativeHistogram - this can be used to build native histograms if one already has the relevant data without calling the observe function. A viable use case is while converting exponential histograms to nativehistograms.

shivanthzen · 2024-10-16T12:47:35Z

@ArthurSens Maybe you can take a look at this PR.

ArthurSens

Thanks for working on this! I've added a few comments but I'm not a native histogram expert, it would be nice if @beorn7 or @krajorama could also take a look here.

question: Have you tried using your commit on OTel's side to make sure you can translate their format into ours?

prometheus/histogram.go

prometheus/histogram_test.go

prometheus/histogram.go

beorn7 · 2024-10-17T16:09:31Z

Will have a look ASAP (which won't be before next week, sorry).

krajorama

Hi, I understand the use case of wanting to expose OTEL exponential histograms from the Prometheus exporter in the collector. How would you use it actually? Constantly register and unregister the metric to update the exposed values? Or just use it's Write() method to get to the data?

In any case, the naming is going to be weird as you're not actually using it as a const :)

If I understand correctly, currently it will only work for counter histograms, not gauge histograms due to limitations in client_golang.

I'm a little afraid of abuse of this interface, but don't see true stoppers for now, will consult @beorn7 .

prometheus/histogram.go

krajorama · 2024-10-21T07:28:34Z

prometheus/histogram.go

+	zeroBucket      uint64
+}
+
+func NewConstNativeHistogram(desc *Desc, count uint64, sum float64, postiveBuckets, negativeBuckets map[int]int64, zeroBucket uint64,


Needs doc string to explain the arguments and the fact that this is exposed as a counter type histogram, treated as a counter on scrape.

In fact, I do actually think that exposing a gauge histogram will be one of the common use cases for using this function. So let's maybe add a bool argument here so that the user can switch between counter and gauge histogram.

Style nit: Prometheus code usually does a line break after the opening parenthesis in case of multiline argument lists.

Tried this. Wouldn't this blow up the argument list? Is it better to create a seperate a constNaticveGuageHistogram ?

I'm still unclear about including a gauge histogram . As per my understanding , we will need to add postiveBuckets, negativeBuckets of type map[int]float64 as an argument (or use a generic parameter). Adding a boolean parameter will not suffice.

It sounds like you are thinking about a float histogram. That's something we might want to add later (ideally together with the option of creating classic float histograms). However, the use case for float histograms in instrumentation is very rare.

I am in fact talking about a gauge histogram. It merely requires to set a different MetricType in the MetricFamily. However, I've just realized that we currently have no way of communicating the difference of counter histogram (i.e. the "normal" histogram) and gauge histogram during metrics collection. For that, we needed Desc to have a metric type field (which was the plan for quite some time).

So I'm pretty sure we cannot solve this problem in this PR, because it needed too much re-architecting.

But if we assume that the difference between counter histogram and gauge histogram will be in the Desc, we actually don't need a bool parameter here, but just have to later allow the option of creating "typed" Descs.

In conclusion, let's not do gauge histograms right now.

That's true. I was indeed confused about a float histogram and gauge histogram.

beorn7

Thanks @shivanthzen for tackling this, and thanks for the review so far @krajorama and @ArthurSens . If I haven't commented on a previous review comment, I agree with it. :)

beorn7 · 2024-10-22T17:20:00Z

prometheus/histogram.go

+	zeroBucket      uint64
+}
+
+func NewConstNativeHistogram(desc *Desc, count uint64, sum float64, postiveBuckets, negativeBuckets map[int]int64, zeroBucket uint64,


In fact, I do actually think that exposing a gauge histogram will be one of the common use cases for using this function. So let's maybe add a bool argument here so that the user can switch between counter and gauge histogram.

beorn7 · 2024-10-22T17:21:26Z

prometheus/histogram.go

+	zeroBucket      uint64
+}
+
+func NewConstNativeHistogram(desc *Desc, count uint64, sum float64, postiveBuckets, negativeBuckets map[int]int64, zeroBucket uint64,


Style nit: Prometheus code usually does a line break after the opening parenthesis in case of multiline argument lists.

prometheus/histogram_test.go

prometheus/histogram.go

shivanthzen · 2024-10-24T19:09:05Z

Thanks for the comments. I'll resolve them this weekend

ArthurSens

I had just a quick look today :)

prometheus/histogram.go

ArthurSens · 2024-10-28T22:13:36Z

prometheus/histogram.go

+	if err := validateCount(count, negativeBuckets, positiveBuckets, zeroBucket); err != nil {
+		return nil, err
+	}


Do we need to validate Sum as well?

Not really. Sum could by anything. I mean, there could be consistency checks like the sum cannot be negative if there are only positive observations, or the sum has to be zero without any observations, and many more. But I don't know if that would be very helpful. We should only check for conditions that would make Prometheus reject the histogram upon ingestion.

ArthurSens · 2024-10-28T22:14:46Z

prometheus/histogram.go

+	return nil
+}
+
+func makeBucketsAny(buckets map[int]int64) ([]*dto.BucketSpan, []int64) {


hey, could you clarify what Any means here?

Probably "positive or negative".
I would just call it makeBuckets.

+1 for makeBuckets

Hmm, makebuckets alreadyexists in the namespace. I could do makeBucketfrommap

beorn7

Looks good in general.

Despite the frequent nit pickings about naming etc., I haven't yet checked the technical correctness very deeply yet (although it looks fine as far as I can see right now).

prometheus/histogram.go

beorn7 · 2024-10-29T15:19:42Z

prometheus/histogram.go

+	if err := validateCount(count, negativeBuckets, positiveBuckets, zeroBucket); err != nil {
+		return nil, err
+	}


Not really. Sum could by anything. I mean, there could be consistency checks like the sum cannot be negative if there are only positive observations, or the sum has to be zero without any observations, and many more. But I don't know if that would be very helpful. We should only check for conditions that would make Prometheus reject the histogram upon ingestion.

beorn7 · 2024-10-29T15:20:57Z

prometheus/histogram.go

+	return nil
+}
+
+func makeBucketsAny(buckets map[int]int64) ([]*dto.BucketSpan, []int64) {


Probably "positive or negative".
I would just call it makeBuckets.

prometheus/histogram.go

ArthurSens

LGTM, but let's wait for an approval from Beorn or Krajo as well

Thanks for the work!

ArthurSens · 2024-10-30T13:11:12Z

Oh, looks like some commits weren't signed and there's one test failing 🤔

The test seems to be flaky, it's also failing in other PRs

shivanthzen · 2024-11-01T13:42:04Z

I can sign the commits once the PR is approved, It seems signing will need a force push

beorn7 · 2024-11-02T12:12:40Z

I can sign the commits once the PR is approved, It seems signing will need a force push

Yes, it changes the commits and therefore needs a force-push. However, it is essential to sign before approval. That's kind of the point.

beorn7

Thanks for addressing the feedback.

I have no realized something about exemplars we had not discussed before.

I have also realized we cannot do gauge histograms right now. Needs more discussion and planning, which is out of scope of this PR.

prometheus/histogram.go

prometheus/histogram_test.go

shivanthzen · 2024-11-03T19:42:32Z

Force pushed with signed commits.

ArthurSens · 2024-11-04T21:00:31Z

@shivanthzen, looks like you accidentally added unrelated commits to your PR, could you keep only the ones that you've authored here?

shivanthzen · 2024-11-06T07:29:15Z

@ArthurSens Removed other commits from the branch

krajorama

couple of minor comments

prometheus/histogram.go

kakkoyun · 2024-11-11T14:32:40Z

@beorn7 could you have another look at it?

beorn7 · 2024-11-12T15:57:24Z

This is very close to the top of my review queue now. Will have a look soon, hopefully today.

beorn7

Just some minor things about doc comments left.

prometheus/histogram.go

beorn7 · 2024-11-13T10:34:04Z

Looks good to me, but there are lint warnings now. (They are coming from a recently added check, but they are still relevant.) Also, DCO signing is missing. @shivanthzen could you address the lint warnings and sign the commits? (Feel free to squash commits into fewer or just one before doing that.)

Signed-off-by: Shivanth <[email protected]> Apply suggestions from code review Co-authored-by: Arthur Silva Sens <[email protected]> Signed-off-by: Shivanth <[email protected]> Fix references and existing tests Signed-off-by: Shivanth <[email protected]> Review comments Signed-off-by: Shivanth <[email protected]> fmt Signed-off-by: Shivanth <[email protected]> validation Signed-off-by: Shivanth <[email protected]> Count validation Signed-off-by: Shivanth <[email protected]> Validation Signed-off-by: Shivanth <[email protected]> Review comments Signed-off-by: Shivanth <[email protected]> Space formatting Signed-off-by: Shivanth <[email protected]> Review comments Signed-off-by: Shivanth <[email protected]> Rename SyncMaptoMap -> SyncMapToMap Signed-off-by: Shivanth <[email protected]> Remove exemplars from parameters for constNativeHistogram function Signed-off-by: Shivanth <[email protected]> Update prometheus/histogram.go Co-authored-by: George Krajcsovits <[email protected]> Signed-off-by: Shivanth MP <[email protected]> Update prometheus/histogram.go Co-authored-by: George Krajcsovits <[email protected]> Signed-off-by: Shivanth MP <[email protected]> Update prometheus/histogram.go Co-authored-by: George Krajcsovits <[email protected]> Signed-off-by: Shivanth MP <[email protected]> Update prometheus/histogram.go Co-authored-by: Björn Rabenstein <[email protected]> Signed-off-by: Shivanth MP <[email protected]> Review comments Signed-off-by: Shivanth <[email protected]> Lint fix Signed-off-by: Shivanth <[email protected]>

…) (#7016) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) | require | minor | `v1.20.5` -> `v1.21.0` | --- ### Release Notes <details> <summary>prometheus/client_golang (github.com/prometheus/client_golang)</summary> ### [`v1.21.0`](https://github.com/prometheus/client_golang/releases/tag/v1.21.0): / 2025-02-19 [Compare Source](prometheus/client_golang@v1.20.5...v1.21.0) :warning: This release contains potential breaking change if you upgrade `github.com/prometheus/common` to 0.62+ together with client_golang (and depend on the strict, legacy validation for the label names). New common version [changes `model.NameValidationScheme` global variable](prometheus/common#724), which relaxes the validation of label names and metric name, allowing all UTF-8 characters. Typically, this should not break any user, unless your test or usage expects strict certain names to panic/fail on client_golang metric registration, gathering or scrape. In case of problems change `model.NameValidationScheme` to old `model.LegacyValidation` value in your project `init` function. ⚠️ - \[BUGFIX] gocollector: Fix help message for runtime/metric metrics. [#1583](prometheus/client_golang#1583) - \[BUGFIX] prometheus: Fix `Desc.String()` method for no labels case. [#1687](prometheus/client_golang#1687) - \[PERF] prometheus: Optimize popular `prometheus.BuildFQName` function; now up to 30% faster. [#1665](prometheus/client_golang#1665) - \[PERF] prometheus: Optimize `Inc`, `Add` and `Observe` cumulative metrics; now up to 50% faster under high concurrent contention. [#1661](prometheus/client_golang#1661) - \[CHANGE] Upgrade prometheus/common to 0.62.0 which changes `model.NameValidationScheme` global variable. [#1712](prometheus/client_golang#1712) - \[CHANGE] Add support for Go 1.23. [#1602](prometheus/client_golang#1602) - \[FEATURE] process_collector: Add support for Darwin systems. [#1600](prometheus/client_golang#1600) [#1616](prometheus/client_golang#1616) [#1625](prometheus/client_golang#1625) [#1675](prometheus/client_golang#1675) [#1715](prometheus/client_golang#1715) - \[FEATURE] api: Add ability to invoke `CloseIdleConnections` on api.Client using `api.Client.(CloseIdler).CloseIdleConnections()` casting. [#1513](prometheus/client_golang#1513) - \[FEATURE] promhttp: Add `promhttp.HandlerOpts.EnableOpenMetricsTextCreatedSamples` option to create OpenMetrics \_created lines. Not recommended unless you want to use opt-in Created Timestamp feature. Community works on OpenMetrics 2.0 format that should make those lines obsolete (they increase cardinality significantly). [#1408](prometheus/client_golang#1408) - \[FEATURE] prometheus: Add `NewConstNativeHistogram` function. [#1654](prometheus/client_golang#1654) <details> <summary> All commits </summary> * Merge release-1.20 to main by @bwplotka in prometheus/client_golang#1582 * gocollector: Tiny fix for help message with runtime/metrics source. by @bwplotka in prometheus/client_golang#1583 * ci: bump dagger to the latest version by @marcosnils in prometheus/client_golang#1588 * Merge release-1.20 back to main by @ArthurSens in prometheus/client_golang#1593 * Update linting by @SuperQ in prometheus/client_golang#1603 * Update supported Go versions by @SuperQ in prometheus/client_golang#1602 * build(deps): bump golang.org/x/sys from 0.22.0 to 0.24.0 by @dependabot in prometheus/client_golang#1611 * build(deps): bump github.com/prometheus/common from 0.55.0 to 0.57.0 by @dependabot in prometheus/client_golang#1612 * changed the name of all variables with min/max name by @parthlaw in prometheus/client_golang#1606 * Update Dagger and build. by @SuperQ in prometheus/client_golang#1610 * build(deps): bump github/codeql-action from 3.25.15 to 3.26.6 in the github-actions group across 1 directory by @dependabot in prometheus/client_golang#1614 * examples: Improved GoCollector example. by @bwplotka in prometheus/client_golang#1589 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1615 * process_collector: fill in most statistics on macOS by @mharbison72 in prometheus/client_golang#1600 * ⚡ http client defer CloseIdleConnections by @cuisongliu in prometheus/client_golang#1513 * Set allow-utf-8 in Format during tests to avoid escaping. by @ywwg in prometheus/client_golang#1618 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1622 * Merge Release 1.20 back to main by @ArthurSens in prometheus/client_golang#1627 * examples: Add custom labels example by @ying-jeanne in prometheus/client_golang#1626 * Refactor default runtime metrics tests for Go collector so that default runtime metric set autogenerates by @vesari in prometheus/client_golang#1631 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1628 * process_xxx_memory statistics for macOS (cgo) by @mharbison72 in prometheus/client_golang#1616 * build(deps): bump github.com/klauspost/compress from 1.17.9 to 1.17.10 by @dependabot in prometheus/client_golang#1633 * build(deps): bump golang.org/x/sys from 0.24.0 to 0.25.0 by @dependabot in prometheus/client_golang#1632 * process_collector: Add Platform-Specific Describe for processCollector by @ying-jeanne in prometheus/client_golang#1625 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1635 * build(deps): bump the github-actions group with 4 updates by @dependabot in prometheus/client_golang#1634 * Optionally print OM created lines by @ArthurSens in prometheus/client_golang#1408 * process_collector: merge wasip1 and js into a single implementation by @ying-jeanne in prometheus/client_golang#1644 * Merge release 1.20 to main by @bwplotka in prometheus/client_golang#1647 * Add Arianna as maintainer 💪 by @ArthurSens in prometheus/client_golang#1651 * test add headers round tripper by @Manask322 in prometheus/client_golang#1657 * build(deps): bump github.com/klauspost/compress from 1.17.10 to 1.17.11 by @dependabot in prometheus/client_golang#1668 * build(deps): bump golang.org/x/sys from 0.25.0 to 0.26.0 by @dependabot in prometheus/client_golang#1669 * build(deps): bump github.com/prometheus/common from 0.59.1 to 0.60.1 by @dependabot in prometheus/client_golang#1667 * build(deps): bump google.golang.org/protobuf from 1.34.2 to 1.35.1 by @dependabot in prometheus/client_golang#1670 * Optimize BuildFQName function by @jkroepke in prometheus/client_golang#1665 * fix: use injected now() instead of time.Now() in summary methods by @imorph in prometheus/client_golang#1672 * process_collector: avoid a compiler warning on macOS (fixes #1660) by @mharbison72 in prometheus/client_golang#1675 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1674 * build(deps): bump the github-actions group across 1 directory with 3 updates by @dependabot in prometheus/client_golang#1678 * [chore]: enable perfsprint linter by @mmorel-35 in prometheus/client_golang#1676 * Duplicate of #1662 by @imorph in prometheus/client_golang#1673 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1679 * chore: enable usestdlibvars linter by @mmorel-35 in prometheus/client_golang#1680 * Add: exponential backoff for CAS operations on floats by @imorph in prometheus/client_golang#1661 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1683 * [1617] Add ConstnativeHistogram by @shivanthzen in prometheus/client_golang#1654 * fix: replace fmt.Errorf with errors.New by @kakkoyun in prometheus/client_golang#1689 * Add codeowners by @kakkoyun in prometheus/client_golang#1688 * fix: add very small delay between observations in `TestHistogramAtomicObserve` by @imorph in prometheus/client_golang#1691 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1692 * Fix: handle nil variableLabels in Desc.String() method and add tests for nil label values by @kakkoyun in prometheus/client_golang#1687 * examples: Follow best practices and established naming conventions by @lilic in prometheus/client_golang#1650 * setup OSSF Scorecard workflow by @mmorel-35 in prometheus/client_golang#1432 * build(deps): bump google.golang.org/protobuf from 1.35.1 to 1.35.2 by @dependabot in prometheus/client_golang#1697 * build(deps): bump golang.org/x/sys from 0.26.0 to 0.27.0 by @dependabot in prometheus/client_golang#1696 * build(deps): bump the github-actions group with 5 updates by @dependabot in prometheus/client_golang#1695 * update links to openmetrics to reference the v1.0.0 release by @dashpole in prometheus/client_golang#1699 * build(deps): bump google.golang.org/protobuf from 1.35.2 to 1.36.1 by @dependabot in prometheus/client_golang#1706 * build(deps): bump golang.org/x/sys from 0.27.0 to 0.28.0 by @dependabot in prometheus/client_golang#1705 * build(deps): bump the github-actions group with 5 updates by @dependabot in prometheus/client_golang#1707 * build(deps): bump github.com/prometheus/common from 0.60.1 to 0.61.0 by @dependabot in prometheus/client_golang#1704 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1703 * Synchronize common files from prometheus/prometheus by @prombot in prometheus/client_golang#1708 * Upgrade to prometheus/common 0.62.0 with breaking change by @bwplotka in prometheus/client_golang#1712 * build(deps): bump golang.org/x/net from 0.26.0 to 0.33.0 in /tutorials/whatsup by @dependabot in prometheus/client_golang#1713 * docs: Add RELEASE.md for the release process by @kakkoyun in prometheus/client_golang#1690 * tutorials/whatsup: Updated deps by @bwplotka in prometheus/client_golang#1716 * process collector: Fixed pedantic registry failures on darwin with cgo. by @bwplotka in prometheus/client_golang#1715 * Revert "ci: daggerize test and lint pipelines (#1534)" by @bwplotka in prometheus/client_golang#1717 * Cut 1.21.0-rc.0 by @bwplotka in prometheus/client_golang#1718 * Cut 1.21 by @bwplotka in prometheus/client_golang#1737 </details> #### New Contributors * @parthlaw made their first contribution in prometheus/client_golang#1606 * @mharbison72 made their first contribution in prometheus/client_golang#1600 * @cuisongliu made their first contribution in prometheus/client_golang#1513 * @ying-jeanne made their first contribution in prometheus/client_golang#1626 * @Manask322 made their first contribution in prometheus/client_golang#1657 * @jkroepke made their first contribution in prometheus/client_golang#1665 * @imorph made their first contribution in prometheus/client_golang#1672 * @mmorel-35 made their first contribution in prometheus/client_golang#1676 * @shivanthzen made their first contribution in prometheus/client_golang#1654 * @dashpole made their first contribution in prometheus/client_golang#1699 **Full Changelog**: prometheus/client_golang@v1.20.5...v1.21.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "* 0-3 * * *" (UTC), Automerge - "* 0-3 * * *" (UTC). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/7016 Reviewed-by: Gusted <[email protected]> Co-authored-by: Renovate Bot <[email protected]> Co-committed-by: Renovate Bot <[email protected]>

shivanthzen force-pushed the constNativeHistogram branch from 85c4d89 to 473720d Compare October 16, 2024 12:31

ArthurSens reviewed Oct 17, 2024

View reviewed changes

shivanthzen force-pushed the constNativeHistogram branch 2 times, most recently from 96eeddd to ad21924 Compare October 18, 2024 10:04

krajorama reviewed Oct 21, 2024

View reviewed changes

beorn7 requested changes Oct 22, 2024

View reviewed changes

ArthurSens reviewed Oct 28, 2024

View reviewed changes

beorn7 requested changes Oct 29, 2024

View reviewed changes

ArthurSens approved these changes Oct 30, 2024

View reviewed changes

beorn7 requested changes Nov 2, 2024

View reviewed changes

shivanthzen force-pushed the constNativeHistogram branch from afbdefd to f900abb Compare November 3, 2024 19:40

shivanthzen force-pushed the constNativeHistogram branch from 356875e to 698a313 Compare November 3, 2024 19:52

shivanthzen force-pushed the constNativeHistogram branch from 698a313 to d7d3022 Compare November 6, 2024 07:26

krajorama reviewed Nov 8, 2024

View reviewed changes

prometheus/histogram.go Outdated Show resolved Hide resolved

prometheus/histogram.go Outdated Show resolved Hide resolved

prometheus/histogram.go Show resolved Hide resolved

prometheus/histogram.go Outdated Show resolved Hide resolved

kakkoyun requested a review from beorn7 November 11, 2024 14:32

kakkoyun added this to the v1.21.0 milestone Nov 11, 2024

beorn7 requested changes Nov 12, 2024

View reviewed changes

prometheus/histogram.go Show resolved Hide resolved

prometheus/histogram.go Outdated Show resolved Hide resolved

prometheus/histogram.go Show resolved Hide resolved

This was referenced Feb 23, 2025

fix(deps): update module github.com/prometheus/client_golang to v1.21.0 RedHatInsights/payload-tracker-go#254

Merged

fix(deps): update module github.com/prometheus/client_golang to v1.21.0 RedHatInsights/module-update-router#99

Open

renovate bot mentioned this pull request Feb 23, 2025

chore(deps): update github.com/prometheus/client_golang to v1.21.0 vareversat/digicert_exporter#125

Open

1 task

red-hat-konflux bot mentioned this pull request Feb 24, 2025

fix(deps): update module github.com/prometheus/client_golang to v1.21.0 (backplane-2.5) stolostron/discovery#607

Closed

1 task

red-hat-konflux bot mentioned this pull request Feb 24, 2025

Update module github.com/prometheus/client_golang to v1.21.0 stolostron/insights-metrics#178

Open

1 task

soerenschneider mentioned this pull request Feb 24, 2025

chore(deps): update module github.com/prometheus/client_golang to v1.21.0 soerenschneider/sc-agent#82

Open

1 task

renovate-bot mentioned this pull request Feb 25, 2025

Update module github.com/prometheus/client_golang to v1.21.0 backube/volsync#1567

Open

1 task

renovate bot mentioned this pull request Feb 25, 2025

fix(deps): update module github.com/prometheus/client_golang to v1.21.0 open-telemetry/opentelemetry-collector-contrib#38179

Open

1 task

renovate bot mentioned this pull request Feb 25, 2025

fix(deps): update module github.com/prometheus/client_golang to v1.21.0 (main) grafana/loki#16446

Merged

1 task

[1617] Add ConstnativeHistogram #1654

[1617] Add ConstnativeHistogram #1654

Conversation

shivanthzen commented Oct 16, 2024 • edited Loading

shivanthzen commented Oct 16, 2024

ArthurSens left a comment

Choose a reason for hiding this comment

beorn7 commented Oct 17, 2024

krajorama left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shivanthzen Nov 1, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

beorn7 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shivanthzen commented Oct 24, 2024

ArthurSens left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

beorn7 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ArthurSens left a comment

Choose a reason for hiding this comment

ArthurSens commented Oct 30, 2024

shivanthzen commented Nov 1, 2024

beorn7 commented Nov 2, 2024

beorn7 left a comment

Choose a reason for hiding this comment

shivanthzen commented Nov 3, 2024

ArthurSens commented Nov 4, 2024

shivanthzen commented Nov 6, 2024 • edited Loading

krajorama left a comment

Choose a reason for hiding this comment

kakkoyun commented Nov 11, 2024

beorn7 commented Nov 12, 2024

beorn7 left a comment

Choose a reason for hiding this comment

beorn7 commented Nov 13, 2024

shivanthzen commented Oct 16, 2024 •

edited

Loading

shivanthzen Nov 1, 2024 •

edited

Loading

shivanthzen commented Nov 6, 2024 •

edited

Loading