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

apply user defined function #31

Open
ywhcuhk opened this issue Jun 20, 2020 · 2 comments
Open

apply user defined function #31

ywhcuhk opened this issue Jun 20, 2020 · 2 comments

Comments

@ywhcuhk
Copy link

ywhcuhk commented Jun 20, 2020

Thanks for the great package. Any chance we can have a zoo::roll_apply type of thing?

@jasonjfoster
Copy link
Owner

Thanks for the feedback. I don't have plans to support user-defined functions because there's not much value the roll package could add relative to existing implementations. Specifically the advantage of the roll package is fast and efficient computation via online algorithms that are implemented in parallelized C . In contrast, a user-defined function is unable to take advantage of these benefits, e.g. RcppParallel requires code not call R in any fashion (read more here: http://rcppcore.github.io/RcppParallel/#thread_safety). For these reasons, the marginal benefit of another implementation is low.

@ywhcuhk
Copy link
Author

ywhcuhk commented Jul 2, 2020

Point well taken. A thing to consider is that roll can then provide a one stop solution to all the rolling functions. In any case, thanks again for the great package!

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

No branches or pull requests

2 participants