Remove Number.set_hook

Open philippjfr opened this issue 7 years ago • 3 comments

Given that we now have a proper mechanism to watch parameter value changes the set_hook on Number is no longer required. As far as I can tell it was added to add unum unit support for topographica, which we never really properly used or implemented. I'd like to add a deprecation warning asap and remove it in 2.0.

Feb 08 '19 15:02 philippjfr

Ok by me.

Feb 08 '19 15:02 jbednar

There's been no deprecation warning added to set_hook and it is documented. Yet, still a good idea to remove it!

Mar 08 '23 10:03 maximlt

Added a deprecation warning in https://github.com/holoviz/param/issues/692

Apr 12 '23 16:04 maximlt