[GitHub] [incubator-tvm] tmoreau89 commented on issue #4887: [VTA] YoloV3 Support

2020-02-19 Thread GitBox
tmoreau89 commented on issue #4887: [VTA] YoloV3 Support URL: https://github.com/apache/incubator-tvm/pull/4887#issuecomment-588623563 Oh my bad, I read your comment too quickly; I like your suggestion and agree; shortening the filename makes sense!

[GitHub] [incubator-tvm] siju-samuel commented on issue #4788: [FRONTEND][TFLITE]Gather, StridedSlice op support added

2020-02-19 Thread GitBox
siju-samuel commented on issue #4788: [FRONTEND][TFLITE]Gather, StridedSlice op support added URL: https://github.com/apache/incubator-tvm/pull/4788#issuecomment-588621902 @u99127 @FrozenGene @wyc-ruiker Thanks for the review. Could you please recheck once again.

[GitHub] [incubator-tvm] icemelon9 commented on issue #4644: [Relay][AutoTVM] Relay op strategy

2020-02-19 Thread GitBox
icemelon9 commented on issue #4644: [Relay][AutoTVM] Relay op strategy URL: https://github.com/apache/incubator-tvm/pull/4644#issuecomment-588610092 @yzhliu @zhiics @comaniac @masahi Could you review again or approve explicitly? -

[GitHub] [incubator-tvm] wweic commented on issue #4628: [Object] Add String container

2020-02-19 Thread GitBox
wweic commented on issue #4628: [Object] Add String container URL: https://github.com/apache/incubator-tvm/pull/4628#issuecomment-588607759 @tqchen I'm working on the comments :-) Will update soon. This is an automated message

[GitHub] [incubator-tvm] liangfu commented on issue #4887: [VTA] YoloV3 Support

2020-02-19 Thread GitBox
liangfu commented on issue #4887: [VTA] YoloV3 Support URL: https://github.com/apache/incubator-tvm/pull/4887#issuecomment-588581429 Sorry, I meant removing the '_on_vta' suffix in the file names. This is an automated message

[GitHub] [incubator-tvm] comaniac opened a new pull request #4917: Fix doc warning

2020-02-19 Thread GitBox
comaniac opened a new pull request #4917: Fix doc warning URL: https://github.com/apache/incubator-tvm/pull/4917 Based on #4908, this is the first PR to remove all doc warnings from Python doc strings. After this fix, total number of doc warnings is reduced to 103 on my local. ---

[GitHub] [incubator-tvm] were commented on issue #4845: [DEV] TVM v0.7 Roadmap

2020-02-19 Thread GitBox
were commented on issue #4845: [DEV] TVM v0.7 Roadmap URL: https://github.com/apache/incubator-tvm/issues/4845#issuecomment-588516798 I am currently working on some end-to-end model stuff, and Relay's optimization pass are too slow. Any plans on improving the compilation speed? ---

[GitHub] [incubator-tvm] wpan11nv commented on issue #4916: [Relay] Fix an assertion exposed by loop vectorizer

2020-02-19 Thread GitBox
wpan11nv commented on issue #4916: [Relay] Fix an assertion exposed by loop vectorizer URL: https://github.com/apache/incubator-tvm/pull/4916#issuecomment-588507998 TVM and Halide are the same while vectorizing a select op: https://github.com/apache/incubator-tvm/blob/master/src/tir

[GitHub] [incubator-tvm] wpan11nv opened a new pull request #4916: [Relay] Fix an assertion exposed by loop vectorizer

2020-02-19 Thread GitBox
wpan11nv opened a new pull request #4916: [Relay] Fix an assertion exposed by loop vectorizer URL: https://github.com/apache/incubator-tvm/pull/4916 - Allows uniform conditions for select expressions (the same as halide) exposed by the loop vectorizer. Signed-off-by: Wei Pan

[GitHub] [incubator-tvm] tqchen merged pull request #4912: [REFACTOR] Polish ffi convention.

2020-02-19 Thread GitBox
tqchen merged pull request #4912: [REFACTOR] Polish ffi convention. URL: https://github.com/apache/incubator-tvm/pull/4912 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[incubator-tvm] branch master updated: [REFACTOR] Polish ffi convention. (#4912)

2020-02-19 Thread tqchen
This is an automated email from the ASF dual-hosted git repository. tqchen pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git The following commit(s) were added to refs/heads/master by this push: new 18295b2 [REFACTOR] Polish ffi convention

[GitHub] [incubator-tvm] comaniac commented on issue #4908: [DOCS] Fix Sphinx Warnings

2020-02-19 Thread GitBox
comaniac commented on issue #4908: [DOCS] Fix Sphinx Warnings URL: https://github.com/apache/incubator-tvm/issues/4908#issuecomment-588413828 I could make a PR to fix the indent and format related warnings. This is an automate

[GitHub] [incubator-tvm] videetparekh commented on issue #4843: Emscripten Runtime Error while linking

2020-02-19 Thread GitBox
videetparekh commented on issue #4843: Emscripten Runtime Error while linking URL: https://github.com/apache/incubator-tvm/issues/4843#issuecomment-588402394 Hi Tianqi, Not sure which thread you're referring to. I've managed to compile a Mobilenet TensorFlow model down through TVM to `.

[GitHub] [incubator-tvm] tqchen commented on issue #4845: [DEV] TVM v0.7 Roadmap

2020-02-19 Thread GitBox
tqchen commented on issue #4845: [DEV] TVM v0.7 Roadmap URL: https://github.com/apache/incubator-tvm/issues/4845#issuecomment-588397980 @liangfu auto tensorization depends on on the new IR update, we will send out specific guidelines after we finish the IR refactoring

[GitHub] [incubator-tvm] tqchen commented on issue #4628: [Object] Add String container

2020-02-19 Thread GitBox
tqchen commented on issue #4628: [Object] Add String container URL: https://github.com/apache/incubator-tvm/pull/4628#issuecomment-588394175 ping @wweic This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #4891: [TEST][FLAKY] topi/tests/python/test_topi_sort.py::test_argsort

2020-02-19 Thread GitBox
tqchen commented on a change in pull request #4891: [TEST][FLAKY] topi/tests/python/test_topi_sort.py::test_argsort URL: https://github.com/apache/incubator-tvm/pull/4891#discussion_r381484199 ## File path: topi/tests/python/test_topi_sort.py ## @@ -27,6 +27,12 @@ def veri

[GitHub] [incubator-tvm] tqchen commented on issue #4885: Split node min range is not stringent.

2020-02-19 Thread GitBox
tqchen commented on issue #4885: Split node min range is not stringent. URL: https://github.com/apache/incubator-tvm/pull/4885#issuecomment-588389845 In this case, unfortunately we need to wait until #4890 is fixed This is an

[GitHub] [incubator-tvm] tqchen closed issue #4911: how the file of "module.loadfile_clbin" produce and use it with opencl

2020-02-19 Thread GitBox
tqchen closed issue #4911: how the file of "module.loadfile_clbin" produce and use it with opencl URL: https://github.com/apache/incubator-tvm/issues/4911 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-tvm] tqchen commented on issue #4911: how the file of "module.loadfile_clbin" produce and use it with opencl

2020-02-19 Thread GitBox
tqchen commented on issue #4911: how the file of "module.loadfile_clbin" produce and use it with opencl URL: https://github.com/apache/incubator-tvm/issues/4911#issuecomment-588389159 Please open a new question thread on https://discuss.tvm.ai/ -

[GitHub] [incubator-tvm] tmoreau89 commented on issue #4887: [VTA] YoloV3 Support

2020-02-19 Thread GitBox
tmoreau89 commented on issue #4887: [VTA] YoloV3 Support URL: https://github.com/apache/incubator-tvm/pull/4887#issuecomment-588378731 @liangfu that is correct; we'd need to rename `deploy_vision_on_vta` to `deploy_classification_on_vta`! ---

[GitHub] [incubator-tvm] tqchen commented on issue #4912: [REFACTOR] Polish ffi convention.

2020-02-19 Thread GitBox
tqchen commented on issue #4912: [REFACTOR] Polish ffi convention. URL: https://github.com/apache/incubator-tvm/pull/4912#issuecomment-588376137 @zhiics updated This is an automated message from the Apache Git Service. To res

[GitHub] [incubator-tvm] inadob opened a new pull request #4915: [Frontend][TFLite] Add parser support for 'square' operator

2020-02-19 Thread GitBox
inadob opened a new pull request #4915: [Frontend][TFLite] Add parser support for 'square' operator URL: https://github.com/apache/incubator-tvm/pull/4915 This patch is built on top of #4913 * 'square' is an unary elemwise operator but it's added separately in the parser since there is

[GitHub] [incubator-tvm] wyc-ruiker commented on a change in pull request #4913: [Frontend][TFLite] Fix tests for tflite unary elemwise operations

2020-02-19 Thread GitBox
wyc-ruiker commented on a change in pull request #4913: [Frontend][TFLite] Fix tests for tflite unary elemwise operations URL: https://github.com/apache/incubator-tvm/pull/4913#discussion_r381283571 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -745,24 +

[GitHub] [incubator-tvm] Robeast opened a new issue #4914: [FRONTEND] Tensorflow Op: Resampler, SparseReshape, Atan2, SparseTensorDenseMatMul

2020-02-19 Thread GitBox
Robeast opened a new issue #4914: [FRONTEND] Tensorflow Op: Resampler, SparseReshape, Atan2, SparseTensorDenseMatMul URL: https://github.com/apache/incubator-tvm/issues/4914 Hello there, I'm currently working on an in-house network and I encountered some unsupported Tensorflow opera

[GitHub] [incubator-tvm] inadob opened a new pull request #4913: [Frontend][TFLite] Fix tests for tflite unary elemwise operations

2020-02-19 Thread GitBox
inadob opened a new pull request #4913: [Frontend][TFLite] Fix tests for tflite unary elemwise operations URL: https://github.com/apache/incubator-tvm/pull/4913 Previously these tests were passing just because they weren't run at all - 'test_op' name was not checked properly - the gen

[GitHub] [incubator-tvm] liangfu commented on issue #4845: [DEV] TVM v0.7 Roadmap

2020-02-19 Thread GitBox
liangfu commented on issue #4845: [DEV] TVM v0.7 Roadmap URL: https://github.com/apache/incubator-tvm/issues/4845#issuecomment-588125167 I wonder is there any active effort working on the feature "Auto tensorization", and I'm interested in contributing.

[GitHub] [incubator-tvm] liangfu commented on issue #4887: [VTA] YoloV3 Support

2020-02-19 Thread GitBox
liangfu commented on issue #4887: [VTA] YoloV3 Support URL: https://github.com/apache/incubator-tvm/pull/4887#issuecomment-588119735 Since the python script deploy_vision_on_vta.py exists in the vta directory already, how about renaming those to deploy_classification.py and deploy_detectio