Upgrade PDFs, add labels #75
Conversation
| m = params["m"] | ||
| t = params["t"] | ||
| n = params["n"] | ||
| x = x[0] | ||
| return tsallis_pdf_func(x=x, m=m, t=t, n=n) |
There was a problem hiding this comment.
Just FYI @ikrommyd , that's what 0.20.0 has in it: the params are now given as an argument (therefore, the method could be static!) and the x can now be accessed via the "axis" (i.e. the x is a Data object, but indexing with 0 returns the 0th axis)
The whole PR is just a chore, an upgrade of internals
There was a problem hiding this comment.
but I see this change only in zfit_physics and not in zfit for now
There was a problem hiding this comment.
Partially, the zfit pdfs have all labels by now (which is needed in the plotting) and some also get the params. It really doesn't change a lot, it's more about the API and how it looks for the user, and we're able to make them static. zfit.supports() provides the magic :)
Minor cleanup to upgrade to zfit 0.20.0
Checklist