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

Ultimate Oscillator class is instantiated with hard coded values #114

Closed
fettunch opened this issue Dec 20, 2019 · 4 comments
Closed

Ultimate Oscillator class is instantiated with hard coded values #114

fettunch opened this issue Dec 20, 2019 · 4 comments
Assignees

Comments

@fettunch
Copy link

Hi @bukosabino ,

this library is very good. I am thinking to use it for one of my projects.

In momentum.py class row 611 the UltimateOscillator is instantiated with hard coded values the row:

return UltimateOscillator(
    high=high, low=low, close=close, s=7, m=14, len=28, ws=4.0, wm=2.0, wl=1.0, fillna=fillna).uo()

should be instead:

return UltimateOscillator(
    high=high, low=low, close=close, s=s, m=m, len=len, ws=ws, wm=wm, wl=wl, fillna=fillna).uo()

I can make this change if you want. Let me know.

@fettunch
Copy link
Author

I found other places whit hard-coded values a part from the ones I opened an issue: KeltnerChannel, Awesome Oscillator.

I change the code and I have a branch ready. if you allow me to push it I can create a pull request for this topic.

@fettunch
Copy link
Author

Hi @bukosabino, do you have any news on this thread?

@bukosabino bukosabino self-assigned this Mar 12, 2020
@bukosabino
Copy link
Owner

Thank you @fettunch ,

I will fix asap

@bukosabino
Copy link
Owner

Hi @fettunch ,

All hardcoded values are fixed if you use the last version 0.5.13. To install the last version:

pip install -U ta

Please, let me know if you have more feedback for me. Thanks!

Best,
Dario

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