This is a snippet of code with some IO stripped. RooKeysPdf fails to parameterize the function in 6.28 but the same code works in 6.26. I tried to do a reproducer but for a simple gaussian model both versions behaved ok. Was something changed in the implementation of keyspdf? Or weighted datasets?
thanks for posting here, it’s very important to get feedback on the new releases!
Yes, there was a change in the implementation for ROOT 6.28:
The results that you got with the RooKeysPdf with weighted datasets in 6.26 and earlier were wrong, because the implementation used the numbers of entries in the dataset in places where the sum of weights should have been.
I would consider therefore both results as suspicious: for 6.26 because the implementation for RooKeysPdf based on weighted datasets was wrong, but also the presumably correct 6.28 implementation because it is not much tested yet and you get the evaluation errors.
The evaluation errors could be harmless, but I can’t say that for sure without seeing your model and datasets. If you want me to look into this, would it be possible to share the dataset and the full script so I can exactly reproduce the log?