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

High-order FD time derivative for SINDy #6

Merged
merged 2 commits into from
Aug 30, 2024
Merged

High-order FD time derivative for SINDy #6

merged 2 commits into from
Aug 30, 2024

Conversation

dreamer2368
Copy link
Collaborator

@dreamer2368 dreamer2368 commented Jul 19, 2024

  • fd module provides high-order, summation-by-parts finite-difference operators.
    • This is proven to approximate time derivative better.
  • sindy.compute_sindy_data is replaced with sindy.compute_time_derivative
    • compute_time_derivative returns the time-derivative array with the same shape of the original solution time series.
    • takes additional input for FD type, to specify the FD operator to use.
    • All the routines involving this routine also take the additional input for FD type.

In future, SINDy part should be factored out from BayesianLaSDI class, in order to support various types of SINDy formulation.

@chldkdtn chldkdtn added the RFR Ready for Review label Aug 15, 2024
@dreamer2368 dreamer2368 merged commit 212df7d into main Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants