Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow creation of a KoopmanRegressor object from a Koopman matrix #129

Conversation

sdahdah
Copy link
Member

@sdahdah sdahdah commented Jan 6, 2023

Resolves #125

Proposed Changes

  • Add DataRegressor, which accepts a Koopman matrix in the form of a NumPy array as input.

Checklist

  • Write unit tests
  • Add new estimators to existing scikit-learn compatibility tests
  • Write examples in docstrings
  • Update Sphinx documentation
  • Bump version number and date in setup.py, CITATION.cff, and
    README.rst

@sdahdah sdahdah added the enhancement New feature or request label Jan 6, 2023
@sdahdah sdahdah added this to the v1.2.0 milestone Jan 6, 2023
@sdahdah sdahdah self-assigned this Jan 6, 2023
@sdahdah sdahdah linked an issue Jan 6, 2023 that may be closed by this pull request
@sdahdah sdahdah merged commit 2a7c24a into main Feb 10, 2023
@sdahdah sdahdah deleted the feature/125-allow-creation-of-a-koopmanregressor-object-from-a-koopman-matrix branch February 10, 2023 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow creation of a KoopmanRegressor object from a Koopman matrix
1 participant