Extended Analysis #320
Replies: 2 comments 12 replies
-
I think for cases where we add extended analysis, we can add The base indicator would be:
And would include a simpler result class only containing |
Beta Was this translation helpful? Give feedback.
-
I think another API option here could look more like: # two steps
var results = quotes.GetSma(5);
var analysis = results.Analyze();
# also supports chaining
var analysis = quotes
.GetSma(5)
.Analyze(); This is a bit easier to use IMO and would keep base indicator calculations as separate concerns. Thoughts and feedback? |
Beta Was this translation helpful? Give feedback.
-
For those of you who've been following the development of this library, before the initial v1.0, we had a few indicators that supplied additional analysis in the results; things like "IsBullish" and "IsReversal". Based on feedback from users (#99), we decided to keep the library in a more basic form and excluded additional analysis values.
I'm open to adding these back since they are useful for identifying trade signals and conditions; however, I'd like to do it in a way that does not degrade performance or add complexity to the basic indicators. We'd need to satisfy both groups of users 1) those who want basic, pure, and fast indicators, and 2) those who are willing to sacrifice a bit of performance for having extended information.
My initial thought was to have an entirely separate "Stock.Indicators.Extended" library, but there are probably several ways this could be implemented.
Thoughts?
Beta Was this translation helpful? Give feedback.
All reactions