-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
chandelier_exit #559
Comments
Hi @bibinvargheset, Sure. It can be added. Hopefully someone can convert it to proper method and make a PR so it can be included. 😎 Kind Regards, |
Hey @twopirllc @bibinvargheset , I'm interested in contributing to this issue, so before I start working on it, would you mind sparing your time explaining what the issue is about and pointing me to some resources to get started |
@9gl the code i shared is working, but needs structural changes to be, check psar.py or similar indicator and update to have similar structure and functionality |
@bibinvargheset Sure will do. Could you point me to the file which requires changes |
Hi @9gl, Thanks for your interest in taking this on. 😎 First, make sure you are using the development branch. README $ pip install -U git+https://github.com/twopirllc/pandas-ta.git@development Second, follow Creating a Custom Indicator: The Big 4 to add an indicator and hook it up to the DataFrame Extension. Notes:
That should cover the majority. The indicator itself is not difficult. I can add tests after the PR, so don't worry about those unless you want to try. Hope this helps! Let me know otherwise. Kind Regards, |
Here is a code that i have written for chandelier_exit, if possible add to repo
The text was updated successfully, but these errors were encountered: