[Bug] TA/multi - unclear what --period affects when multiple functions are selected.

[deeleeramone]

The Multi function does not have the ability to set parameters for each indicator, and it is unclear what --period even means where it comes to indicators having multiple parameters. Does it change anything if multi --indicators macd --period 10? The MACD function does not even contain a parameter for period. Unclear how this functionality works.

The loaded ticker is: /stocks/load/ubs

The top chart is: multi --indicators clenow,ema --period 10
The bottom chart is: clenow --period 10

[IgorWounds]

Hi @deeleeramone ,

Thanks for raising this!

The --period is the size of the rolling window that an indicator should use. By default, the rolling window/period is set automatically (window = [20, 50]) if it is not explicitly passed by the user.

The user can also pass multiple periods --period 10,30 that will be picked up by all indicators that support different rolling window sizes and will be displayed.

For example, If I do multi --indicators sma,ema --period 10,30 I get the following:

It seems that some indicators don't complain when you pass a period that isn't used and this might be something that we want to be explicit about. Currently, the code sets the window size for all Plotly indicators that support the length parameter natively. I'm thinking of stating inside the terminal which of the passed indicators don't support the --period flag once the command has been executed.

[deeleeramone]

PR #4467 addresses some of this, so I would advise looking at it to avoid any duplication. :)

[IgorWounds]

From what I see, the explicit message wasn't added inside that PR.

[IgorWounds]

The period flag has been removed inside the #4467 PR and I'll close this now.