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

Added nadaraya-watson channel indicator #698

Closed
wants to merge 1 commit into from

Conversation

tdgrossi
Copy link

Closes #697

Hi! This code will add the nadaraya-watson channel indicator. It was inspired by this, and this, but turned out quite different in how it approaches the calculation. My implementation is more of a "pure" nadaraya-watson regression, and quite frankly, more efficient with less nested loops. In addition, it allows users to use different kernel models to calculate the regressions (Lux's uses the gaussian kernel).
I'm not a mathematician so don't ask me difficult questions.

@twopirllc
Copy link
Owner

Hello @tdgrossi,

Cool! 😎 Would be great if you could checkout the development branch and make a PR for that branch. If not possible, I will do it when I get a chance.

Kind Regards,
KJ

@tdgrossi
Copy link
Author

tdgrossi commented Jun 27, 2023

Sure thing! Will work on it in the next few weeks.

@tdgrossi tdgrossi closed this Jun 27, 2023
@pybymo
Copy link

pybymo commented Jan 2, 2024

Hi, to use this indicator, should I do file changes manually? because my pandas-ta is up to date and there isn't nwc!

@tdgrossi
Copy link
Author

tdgrossi commented Jan 2, 2024

Hi @pybymo , we haven't merged this indicator to the master branch, so it's not part of pandas. Further, mind you that I found out that the indicator as is contains lookahead bias (aka repaints). I do have a working version without repainting, but never got a chance to put it here. Might someday.

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.

Add the Nadaraya-Watson Channels indicator
3 participants