tqchen commented on a change in pull request #6302:
URL: https://github.com/apache/incubator-tvm/pull/6302#discussion_r490326071
##########
File path: python/setup.py
##########
@@ -166,7 +166,19 @@ def get_package_data_files():
],
extras_require={
"test": ["pillow<7", "matplotlib"],
- "extra_feature": ["tornado", "psutil", "xgboost>=1.1.0", "mypy",
"orderedset"],
+ "extra_feature": [
+ "tornado",
+ "psutil",
+ "xgboost>=1.1.0",
+ "mypy",
+ "orderedset",
+ "tensorflow==2.1.0",
+ "tflite==2.1.0",
Review comment:
ideally froma users' PoV, we don't want to restrict the dependency to a
certain version. `>=` is better. Although we understand things are a moving
target. e.g. a user might choose to install a different version of
tensorflow/onnx
We should express the current CI setups in a separate requirement setting.
----------------------------------------------------------------
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]
