-
Notifications
You must be signed in to change notification settings - Fork 24
Description
This has been in the back of my mind, but it came up again because I was thinking it made more sense if, based on #138, we say all the TVR methods are expressions of one method and optimize over the derivative order, optimize calls. Then as part of #143, I have gone and made the currently-private common backing functions for the TVR, Kalman, and Iterated FD modules public, thereby enabling one to optimize a single function for the module. Then this of course necessitated messing with the optimization code, and I realized Iterated FD needs categorical order, because it can be 2 or 4 but not 3, so a numerical parameter that rounds to the nearest int wasn't gonna work.
After some work, categorical parameters are supported, and in the process I've realized it does not actually make sense to try to optimize over