Wrap all StdHook methods

[Wauplin]

Hi !

First of all, thanks a lot for this package and for the Deepkit software in general ! We are using it to share experiments internally in our team and it helps a lot 😃

I encountered an issue when trying to debug my code using pdb. It has to do with the terminal (stdout) that is catched around the StdHook wrapper that logs messages to Deepkit. When I use pdb, I cannot use the arrows to retrieve previous expressions. It is the same issue as described in this issue (even though it's a different package and the source of the problem is different).

I have made a workaround to redirect all calls to StdHook object to the inner object self.s, except for the write method which is the one that needs to be overloaded. Maybe this slight improvement can be useful for other people 🙂

[marcj]

Thanks @Wauplin! :) Which python version do you use? I added comments to the changes itself, as I'm not sure about the fix itself yet.

[marcj]

shouldn't this have a return statement? Or what is this supposed to do? super(StdHook) doesn't look right to me since StdHook has no parent class? I'm not sure I understand this fix.

return super(StdHook, self).__getattr__(name)

[Wauplin]

Hi @marcj, thanks for the quick feedback

Which python version do you use?

I'm using Python 3.8.10 and I have to admit, I didn't try any other version.

shouldn't this have a return statement?

Yes, my mistake.

Or what is this supposed to do? super(StdHook) doesn't look right to me since StdHook has no parent class?

I have changed a bit the fix and added some comments with some references to explain basically what's happening. In fact, super(StdHook) is just returning the base class object which implements the base __setattr__.