Skip to content

Address code to be removed in 6.44 - #23892

Open
vepadulano wants to merge 2 commits into
root-project:masterfrom
vepadulano:fix_further_warnings
Open

vepadulano wants to merge 2 commits into
root-project:masterfrom
vepadulano:fix_further_warnings

Conversation

@vepadulano

Copy link
Copy Markdown
Member

This is a continuation of #23881 which was closed prematurely.

This one introduces further fixes related to RooFit (included in the release notes).

@guitargeek as discussed offline, I am not including the changes related to the removal of the legacy evaluation mode, given that those would be more involved. With a local build of ROOT I don't see warnings related to that, but let's see if the CI agrees here.

@pcanal @jblomer there is still one I/O related class scheduled for removal in 6.44,

} R__DEPRECATED(6, 44,
. In a local build of ROOT I don't see the deprecation warnings. Let's see if we can leave it out of this PR in the interest of restoring a working CI.

* Remove support for implicit Python set to RooArgSet conversion.
* Remove RooTrace.
* Remove BatchMode command argument.
Introduces release notes document for 6.44. Adds removed features
related to I/O, Python, RooFit.
@vepadulano vepadulano self-assigned this Oct 29, 2026
@vepadulano vepadulano added in:Build System clean build Ask CI to do non-incremental build on PR labels Oct 29, 2026

@guitargeek guitargeek left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for taking care of this

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR in:Build System

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants