Closed kuke closed 3 months ago
@kuke I think we can collect all the ops needed to be converted required by the listed models.
It's better to do Design the unit test framework to test every operator's conversion
first and we can implement converter for fluid ops parallel. When all operator converter finished, I think it would be easy to validate the models.
Yeah, I agree with @pkuyym . I think it might be better to do design the unit test framework for a variety of operators.
I also agree with that unit test is very important, so I assign it to myself. But if you are interested in verifying the correctness of models' conversion, you can work on one now. From my experience, a runnable model will help you test the related operator's conversion and the unit test is not necessary.
I'll take READMEs/installation, etc. Thanks @kuke and @pkuyym for your work on the models.
@varunarora Great! It is the time to have a friendly README and installation. I am going to continue to work on models' conversion. I think rest of us can pick up a model or general operators' conversion. You see we have a lot of operators waiting to be implemented the conversion.
We are going to support the conversion of models as follows at first stage. They all can be found in our models bank, some maybe only need to be verified after #19 merged, and some lack of necessary operators.
Even all the models above are supported, only a small subset of operators are used and verified. So for the rest operators, we may need another task:
Hints: