Implement `summary` method for `cp.pymc_experiments.InstrumentalVariable`

[drbenvincent]

At the moment cp.pymc_experiments.InstrumentalVariable does not have an implementation of the summary method. It would be good to add this, and to add test coverage (e.g. in test_iv_reg in test_integration_pymc_examples.py)

Tagging @NathanielF - feel free to assign yourself if you want to take it up :)

[HPCurtis]

@NathanielF have you made a start with this? I would also be happy to give it a crack.

[NathanielF]

Hi @HPCurtis

I have not even made a start at it!

Feel free to do so! And thank you!

[HPCurtis]

@drbenvincent @NathanielF. When following how the summary method is implemented in the other experiments.py classes, we encounter an error related to how print_coefficients() is implemented in the PyMCModel class within pymc_models.py. This error arises because the method expects model beta parameters to be named exactly as "beta". However, in the InstrumentalVariable class, there needs to be a distinction between beta_t and beta_z. Advice on how to proceed would be helpful. For example, should I write an InstrumentalVariable-specific print_coefficients() method, or should I implement logic to handle the specific IV case within the existing method?

[drbenvincent]

Looking at the UML diagram, I would say it makes most sense to override the base print_coefficients method in the InstrumentalVariable class

Reminder that this will affect the API docs, so remember to try to build the docs locally to check it's all as expected.

Thanks for taking it on 👍🏻