Skip to content

Improve RInterface::HistoND signature with an extra argument for weight #20816

@vepadulano

Description

@vepadulano

Explain what you would like to see improved and how.

According to the HistoND documentation, we are currently accepting a list of columns where the size matters:

/// (N columns for unweighted filling, or N+1 columns for weighted filling)

This should instead be aligned with other histogram actions signature, where the weight is a separate argument, e.g. for Histo3D

RResultPtr<::TH3D> Histo3D(const TH3DModel &model, std::string_view v1Name, std::string_view v2Name,
std::string_view v3Name, std::string_view wName)

Extracting this comment into a separate issue to keep better track of it #20664 (comment)

ROOT version

Any

Installation method

Any

Operating system

Any

Additional context

No response

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions