Find instances of rv.logp(x) in PyMC and correct to pm.logp(rv,x)

[canyon289]

Description of your problem

Scan all docs, including pymc-examples for rv.logp() signature and relpace with pm.logp(rv, x)

See #5048 for an example

[ricardoV94]

Btw this also applies to .logcdf and to a lesser extent .random.

[DebadityaPal]

Does this issue still need to be worked upon? if so, I'd like to do it.

[ricardoV94]

Does this issue still need to be worked upon? if so, I'd like to do it.

Very late reply, but yes I think it still needs to be worked out.

[mjhajharia]

@canyon289 are we maintaining backward compatibility for v3 users? if not, maybe a warning or something?

[canyon289]

We're not maintaining full backwards compatibility. There was another PR merged that made sure things didnt totally break, but this issue isnt about that.

This PR is to update references to the new usage pattern. Hope that clarifies things

[mjhajharia]

We're not maintaining full backwards compatibility. There was another PR merged that made sure things didnt totally break, but this issue isnt about that.

This PR is to update references to the new usage pattern. Hope that clarifies things

gotcha, thanks ravin! as ricardo pointed out the other api changes we need to update, we should probably make a checklist from the release notes maybe?

[canyon289]

Checklist or issues, whatever works best for the team

[LucyJimenez]

I would like to work on this one.

[canyon289]

@LucyJimenez Assigned to you! Thank you and let us know if you need help