jwfromm commented on pull request #7823:
URL: https://github.com/apache/tvm/pull/7823#issuecomment-818163952


   @comaniac, the OOP approach was something we discussed and are on the fence 
about implementing. The reason we went with this functional API is because we 
want this API to be catered to new users trying to understand TVM. These users 
will often not be familiar with ML deployment and breaking the process down 
into 4 concrete steps (load, tune, compile, and run) can help crystallize 
exactly what TVM is doing. I feel that making it object oriented instead 
somewhat obscures the steps. That said, I do agree that there are benefits to 
OOP such as discoverability and would be fine adding class methods if you feel 
strongly that its better.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to