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

Update extrema.R #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

chuckcoleman
Copy link

Added xts support for extrema.R.

@helske
Copy link
Owner

helske commented Dec 7, 2021

Thanks, all the functions in Rlibeemd support ts class, but it could be useful to have support for other time series objects such as xts as well. However, I think it would then make sense to add that support to all functions i.e. emd, eemd, ceemdan and bemd. If you add those I'll gladly merge.

@chuckcoleman
Copy link
Author

chuckcoleman commented Dec 7, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants