Alan Kaptanoglu
Alan Kaptanoglu
Yeah, this is a good idea in general to add some docstrings. That ODE in particular was added because it is what Kardy uses to showcase SINDy-PI. Unfortunately there are...
On this note, I have an undergraduate working with the dysts database (https://github.com/williamgilpin/dysts) which would be a really nice database to integrate better with PySINDy for users that want to...
Totally agree with the general sentiment that the code could use a major reorganization because there are a lot of historical hacks. On the trapping SINDy side of things, it...
Couple other things: 1. What is the "parallel" problem you're referring to? I don't think we have anything parallelized in the code. 2. The weak form is incompatible with any...
A reminder to make these changes on a separate branch and then to PR it. Couple other thoughts: 1. By discrete SINDy, you mean fitting equations looking like e.g. x_{k+1}...
@Jacob-Stevens-Haas there is a small error in the constraints -- but I think we fixed this on @MPeng5 's trapping branch of the pysindy code and have been meaning to...
Sorry it took me a while to get to this. Remember that "u" here is the fluid velocity, not the fluid position, so there is no need to have u_dot...
@lankef Frank, tagging you here in case you haven't seen this pull request and its helpful for you
Agreed that it would be great to get out of the differentiation business, and also agreed with Zack that it would be great to retain the fast finite differences. Happy...
One extra note: sometimes I want to show visually that the integrator blew up. solve_ivp doesn't output things if it fails/blows up, but odeint does. This is really nice for...