Addition of Intraday Momentum Index (IMI) indicator

[delfoxav]

I did my best to follow your nomenclature and structure. I checked and the indicator can be loaded in a strategy without any issue. I let you review it.

Kind Regards,

Xavier

[twopirllc]

Hello @delfoxav

Great! But, if you want it included, you would need to base it off the development branch. That is where all bug fixes, features and indicators happen.

Kind Regards,
KJ

[delfoxav]

Oh I see, I forked main in fact, didn't see your dev branch. Seems like I got some conflicts when trying to pull to your dev. I can probably refork your dev and deal with the conflicts on my end before submitting you another pull request.

Best,

Xavier

[twopirllc]

@delfoxav

Oh I see, I forked main in fact, didn't see your dev branch.

No worries.

... I can probably refork your dev and deal with the conflicts on my end before submitting you another pull request.

In general yes. Do look at a few other indicators in the dev branch for the updated coding style. There are better validators and the docstrings have been moved back into the functions.

KJ