hamilton icon indicating copy to clipboard operation
hamilton copied to clipboard

Change node hooks to take in HamiltonNode

Open elijahbenizzy opened this issue 1 year ago • 1 comments

Is your feature request related to a problem? Please describe. StaticValidator does, but the rest don't. We should fix this. We just need to keep it backwards compatible, and we can do this with **future_kwargs. Possibly log a warning as well.

Describe alternatives you've considered Keeping it the way it is

Additional context API consistency

elijahbenizzy avatar Jan 25 '24 14:01 elijahbenizzy

I believe this can be closed?

zilto avatar May 02 '24 16:05 zilto