Add unidirectional sequence lstm

[SebastianBoblest]

This work has mostly been done by @vdkhoi Khoi Duy Vo from ETAS Gmbh.
We add parser support for UnidirectionalSequenceLSTM layers in tflite.

A question regarding the test:
At the moment it uses a toy model that I store in a repo in my github account.
Should we copy this to the TVM repo or what is the best way to do this?

[vdkhoi]

All tests passed

[SebastianBoblest]

Now this fails on windows. I restarted the CI pipeline three times. Does not seem to be related to our changes. Should or could we do anything to resolve this?

[Mousius]

@SebastianBoblestETAS I think this is effecting more than just this PR, I've raised #11220 to track it, please stand by 😸

[huajsj]

code LGTM, thanks @SebastianBoblestETAS, @vdkhoi.

[SebastianBoblest]

@Mousius Hi, sorry this is again failing but in a totally different place now.
Should we simply wait or trigger the CI/CD again?

[SebastianBoblest]

There is still the issue of the model we use in the tests.
Where should we put it?
@huajsj Is your approval not enough for this to get merged?

[Mousius]

@SebastianBoblestETAS I re-ran the CI yesterday and it looks green, though I don't really know that much about this PR other than the CI issues - maybe @leandron can help get this merged? 😸

[SebastianBoblest]

@leandron Hi, could you maybe help with getting this merged?
The issue that still should get resolved however is that our test uses a model that I have stored in a repo in my own github profile.
Other tests seem to use models from public model zoos. But for our case we would prefer a sample model that only has a UnidirectionalSequenceLSTM layer. Should we put it in a data folder in TVM or what would you suggest?

[vdkhoi]

@junrushao1994 Hi, could you also have a look on our PR and help to make it merged? Thanks.

[SebastianBoblest]

@mbrookhart @jwfromm @Huyuwei @hlu1 @AndrewZhaoLuo @kazum @siju-samuel @srkreddy1238 @FrozenGene
Hi all,
could someone of you help us with this PR?
Sorry to bother all of you, I just looked in CONTRIBUTORS.md for all committers that are familiar with frontends.
Thanks in advance!

[AndrewZhaoLuo]

Ill take a look tomorrow

[vdkhoi]

Ill take a look tomorrow

@AndrewZhaoLuo did you review our PR? We are ready to answer your question.

[AndrewZhaoLuo]

Sorry for getting back to you late. LGTM!

[vdkhoi]

@Mousius We just add some comments as request from @AndrewZhaoLuo, but the tvm-ci failed again. Could you please support us to restart it?

[AndrewZhaoLuo]

@vdkhoi you can restart CI by pushing an empty commit.

e.g.
git commit --allow-empty "jostle-ci" and git push

[SebastianBoblest]

@AndrewZhaoLuo I updated the comment in the function and made it more precise.
The most important difference of the two "unbind" versions is actually that the onnx-Version takes a exp.Call object and the one in tflite a tvm.relay.frontend.tflite.TensorWrapper. So inferring the shape also works differently.

[AndrewZhaoLuo]

Sometimes you also need to rebase on main to solve flaky CI issues :/ . Sorry about that

[SebastianBoblest]

@AndrewZhaoLuo I just rebased. Let's hope for the best 😄

[AndrewZhaoLuo]

Thanks! Sorry this took a long time to get merged.

[vdkhoi]

Thanks! Sorry this took a long time to get merged.

Thank you. Finally :) it done