MarisaKirisame commented on issue #4979: [Relay/TOPI/TF][Op] Add support for randomuniform Op in TVM URL: https://github.com/apache/incubator-tvm/pull/4979#issuecomment-593471655 @jroesch @tqchen Having effect in the TVM ir will make optimization on that level harder (because now it has to do effect analysis), and effect do not play nice with Relay's graph representation. I personally think that all TVM operator should be referential transparent, and the effect should happen at the relay level.
---------------------------------------------------------------- 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] With regards, Apache Git Services
