tqchen commented on PR #14094:
URL: https://github.com/apache/tvm/pull/14094#issuecomment-1441102540
In this case, what was being demonstrated was an item x("old interface") in
the overall architecture, which was not scoped under the context of overall
architecture A (including "other older interfaces").
When proposing a new item y here. y may be consistent with x, but it is not
consistent with A, which are also part of the overall architecture. Obviously
there is a tradeoff here.
The ask is that considering the overall architecture, please start with a
way that is consistent with A (where all other modules are properly scoped), it
would indeed introduce inconsistency with x. From the overall project's pov
this tradeoff is the right step forward, and what one would normally recommend.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
