dynamicslab / pysindy

A package for the sparse identification of nonlinear dynamical systems from data
https://pysindy.readthedocs.io/en/latest/
Other
1.42k stars 310 forks source link

[BUG] Is correction necessary when tensoring weak libraries, but not using a ParametrizedLibrary? #361

Open Jacob-Stevens-Haas opened 1 year ago

Jacob-Stevens-Haas commented 1 year ago

The ParametrizedLibrary has a correction term when tensoring weak libraries. Does that code better belong in TensoredLibrary? If I use a TensoredLibrary for weak libraries without correction, would that create a nonsensical optimization problem?

More abstractly, I get the feeling the subclasses ParametrizedLibrary and GeneralizedLibrary largely exist in order to handle tensoring/concatting weak libraries where the TensoredLibrary doesn't handle weak form correctly. If so, and if that were corrected, would we be better without the extra classes?