-
-
Notifications
You must be signed in to change notification settings - Fork 53
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
projection osclliator returns negative values and values greater than 100 PO values range should be [0 -> 100] on node-js environment. #218
Comments
Thank you for filing a bug for it. You are right, it should be returning numbers between 0 and 100. There must be something wrong here. I'll take a look definitely. If you see anything in the code that is not right, please do let me know as well. |
Hi, is this issue resolved or is someone working on it currently? Else I would like to look into this, if its fine. |
Hi Abhishek, Not yet fixed, and no active development on this as of now, so please feel free to take it. |
Since this same issue was also raised in the indicator go repo. I am referring to the same resources that were pointed out there to implement this indicator. Our Codeconst x = generateNumbers(0, closings.length, 1);
const lsHighs = movingLeastSquare(period, x, highs);
const lsLows = movingLeastSquare(period, x, lows);
const vHighs = add(highs, multiply(lsHighs.m, x));
const vLows = add(lows, multiply(lsLows.m, x)); According to the resources the pseudo code looks as follows: Pseudo Code
I think the error is in calculating Possible ErrorIn this pseudo code, they are calculating the indicator value on each incoming bar(represented by Also any suggestions on how I can verify the correct working of the indicator(example: sample data along with correct values of indicator)? |
I tried to compute the projection oscillator for 15min frxGBPUSD candles, the total number of candles were 5000, in number. The projection oscillator returns negative values and values greater than 100 PO values range should be [0 -> 100] . The package function package.projectionOscillator returns an object res = { po:[ items ], spo:[ items ] };
res.po less than 0
res.po greater than 100
res.spo less than 0
res.spo greater than 100
The text was updated successfully, but these errors were encountered: