There is some design work to do with the activation functions and constraints to gain more code reuse and expand the list of supported activations.
This issue originally came from a PR comment by @jalving.
add_constraint=True is not used here
net_block and net are also not used in activation methods. we should discuss whether we need them. they might be there for consistency on the calling side.
There is some design work to do with the activation functions and constraints to gain more code reuse and expand the list of supported activations.
This issue originally came from a PR comment by @jalving.
add_constraint=True is not used here
net_block
andnet
are also not used in activation methods. we should discuss whether we need them. they might be there for consistency on the calling side._Originally posted by @jalving in https://github.com/cog-imperial/OMLT/pull/24#discussion_r784112237_