search

amcharts / amcharts5

The newest, fastest, and most advanced amCharts charting library for JavaScript and TypeScript apps.
Other
353 stars 95 forks source link

Enhancement : Moving average cross indicator #1733

Closed workingbuddy10 closed 1 month ago

workingbuddy10 commented 1 month ago

There is no option of choosing Type and Field in MA cross indictor.

Whereas in Moving Average indicator one can choose the Type and field.

In MA cross the library is directly using simple ma https://github.com/amcharts/amcharts5/blob/1700061a162a22e57cb0c201271add7304243278/src/.internal/charts/stock/indicators/MACross.ts#L133

It would be great if user can get the feasibility to choose/change.

image image

martynasma commented 1 month ago

We have looked into this and came to conclusion that we won't be able to implement this, I'm afraid.

Thanks for the suggestion.

workingbuddy10 commented 1 month ago

Any technical reason ? @martynasma Because I wanted to know the scope of creating the custom indicator like MA cross just giving the Type options. So wanted to know is there any edge case or trivia why you guys are not implementing. So that I can take care of it.

martynasma commented 1 month ago

I'm afraid I can't provide the reasoning behind it. At the moment of making of this indicator it was decided not to have those properties, but I can't work out the reason. Sorry.

workingbuddy10 commented 1 month ago

No issues, Thanks for the help!