jblaschke / call-monitor Goto Github PK

Just to clarify, for the example provided in section Registering your own Argument Handlers, is the last line
REGISTRY.add(np.ndarray, NPHandler)
or
REGISTRY.add(np.ndarray, Handler)?

Here's what I've tried, but not sure what would be the correct usage.

>>> REGISTRY.add(np.ndarray, NPHandler)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'NPHandler' is not defined
>>> REGISTRY.add(np.ndarray, Handler)

Thanks!

Currently the README.md is a little bland on detail. We should add some examples.

Here:

This is not strictly necessary if we don't explicitly call the numpy handler -- but I think this is good manners. Let me ruminate on this a bit further.

Currently only does functions -- a simple fix would be to add the class name to the function name