@salilmishra23 thank you very much for your PR.

I will review it by tomorrow and share feedback.

@lalitpagaria I have added the mypy hook for pre-commit.

Does adding pytest as a hook seem feasible at this point of time? It has to be run after every commit which takes time specially due to downloading of the huge model. It might not be helpful for people making small quick changes.

Anyways let me know what you think about this. I have no other problem if you want me to add pytest hook.

@salilmishra23 You PR is complete functionality wise. Can you please add information about how to perform pre-commit check in contribution guideline. Post it I will approve and merge this PR.

Okay let's skip pytest for now, we can later add when we feel a need for pytest in pre-commit.

Does adding pytest as a hook seem feasible at this point of time? It has to be run after every commit which takes time specially due to downloading of the huge model. It might not be helpful for people making small quick changes.