-
Notifications
You must be signed in to change notification settings - Fork 184
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
MovingAverageConvergenceDivergence and .Macd both unusable #32
Comments
Probably the same issue as #27. It seems the error is caused because the return type is a valuetuple, it's probably a bug of clr as i do not use any unsafe code or external call to com component. I am quite busy now & may not have time to dig deep to the problem but i will try. |
I notice that the longer error in my second example has mention of a MissingMethodException. Maybe it might have to do with the way you set up the return types as you mentioned. Anyway please let me know once you have time to work on this bug and if you need help reproducing it. I tested the MACD method calls (both traditional and extension types) in a sandboxed environment and still got the same errors while other indicator calls such as EMA, RSI, and MA all work fine. |
@mickeyjk could you provide a sample unit test with test data (csv format) to replicate the issue? The smaller the data set, the better and easier to diagnose. |
The issue is no longer active, closed |
I am attempting to utilize the MACD indicators in this library and keep having exceptions thrown whenever the indicator is instantiated or computed. I tried debugging in several different manners to no avail. If necesarry I can post the larger code context but I don't believe that it is the issue since I can create and execute RSI, EMA, and MA indicators with no issue using the same parameters
var tCandles = ToTradyCandles(allCandlesForCoin); var macd = new MovingAverageConvergenceDivergence(tCandles, MA_period_fast, MA_period_slow, signal_period).Compute();
Throws the exception:
var currentValues = tCandles.Macd(MA_period_fast, MA_period_slow, signal_period)[tCandles.Count - 1].Tick;
Throws the exception:
The text was updated successfully, but these errors were encountered: