areusch opened a new pull request #7289: URL: https://github.com/apache/tvm/pull/7289
This PR implements the [Consolidating Python Dependencies RFC](https://discuss.tvm.apache.org/t/rfc-consolidating-tvm-python-dependencies/8329). As part of this RFC, we decided to try a solution halfway between burdening TVM with a full-blown dependency tool such as poetry and the approach we currently take (scattered Python deps). The initial patchset here contains the tool plus some examples of the dependencies. The remaining things needed to merge this PR: - [ ] Write tests for the tool - [ ] Finish consolidating various dependencies from the ci docker containers I'm posting this now before I do any of the test work to get feedback on the approach now that there's something concrete to look at. I do think this tool is a bit more complex than it could be, and am concerned about maintaining it. In particular the logic needed to lint the dependencies list is complex. I don't see a solution that avoids joining a set of dependencies by feature against a constraints list as very workable given the current extras structure. Another topic for discussion is the use of semver. Python does contain a compatibility operator we could use. We should probably debate between the two (let's use the forum for this), as it would remove some complexity here. Comments would be quite welcomed! @tqchen @tkonolige @mshawcroft @leandron @junrushao1994 @tmoreau89 @jwfromm @jroesch @mdw-octoml @zhiics ---------------------------------------------------------------- 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]
