manupa-arm commented on PR #11809: URL: https://github.com/apache/tvm/pull/11809#issuecomment-1168441784
I agree that having modular (smaller) container aids the ease of reproducability from a developer PoV. I'm worried with this approach is that this creates a room for set of diveregent dependencies to be introduced between microTVM vs TVM (the deps that are not specific to micro), especially when they are not seperate projects. Right now, only thing that keeps the common deps align is on the basis that we use the same helper scripts (ubuntu_install*.sh) to construct the image. E.g. : This could open up the possibility to advance the version of tensorflow supported by TVM project while microTVM lacking behind because there will now be tested in two environments. Maybe we could create a base container with such common deps to ensure such divergence are harder to do ? (and extend that image into ci_cpu and ci_qemu) -- 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]
