[GitHub] [incubator-tvm] ANSHUMAN87 commented on a change in pull request #5588: [Frontend][Tensorflow] Gather nd bug fix for one dim support in tensorflow

ANSHUMAN87 commented on a change in pull request #5588: URL: https://github.com/apache/incubator-tvm/pull/5588#discussion_r427070910 ## File path: src/relay/op/tensor/transform.cc ## @@ -2248,6 +2248,9 @@ bool GatherNDRel(const Array& types, int num_inputs, const Attrs& attrs,

[GitHub] [incubator-tvm] icemelon9 commented on pull request #5459: [Relay][Op]Support symbolic TopK, Ones, Zeros and Full

icemelon9 commented on pull request #5459: URL: https://github.com/apache/incubator-tvm/pull/5459#issuecomment-630617284 Could you also update [`IsDataDependant`](https://github.com/apache/incubator-tvm/blob/master/src/relay/analysis/util.cc#L438) function such that the shape func is not d

[GitHub] [incubator-tvm] icemelon9 commented on a change in pull request #5459: [Relay][Op]Support symbolic TopK, Ones, Zeros and Full

icemelon9 commented on a change in pull request #5459: URL: https://github.com/apache/incubator-tvm/pull/5459#discussion_r427061276 ## File path: src/relay/transforms/pattern_util.h ## @@ -325,6 +345,65 @@ inline bool IsEqualScalar(const Expr& a, const Expr& b) { return tvm:

[GitHub] [incubator-tvm] hlu1 commented on pull request #5622: [Replay] Move compiler_begin/end_op to local static objects

hlu1 commented on pull request #5622: URL: https://github.com/apache/incubator-tvm/pull/5622#issuecomment-630613549 @zhiics, @tqchen, please review This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-tvm] hlu1 opened a new pull request #5622: [Replay] Move compiler_begin/end_op to local static objects

hlu1 opened a new pull request #5622: URL: https://github.com/apache/incubator-tvm/pull/5622 Hit compilation error: ``` [op.cc:66] Check failed: reg != nullptr Operator annotation.compiler_begin is not registered ``` It's because `compiler_begin_op` and `compiler_end_op` are non

[GitHub] [incubator-tvm] kevinthesun commented on a change in pull request #5588: [Frontend][Tensorflow] Gather nd bug fix for one dim support in tensorflow

kevinthesun commented on a change in pull request #5588: URL: https://github.com/apache/incubator-tvm/pull/5588#discussion_r427046505 ## File path: src/relay/op/tensor/transform.cc ## @@ -2248,6 +2248,9 @@ bool GatherNDRel(const Array& types, int num_inputs, const Attrs& attrs

[GitHub] [incubator-tvm] kazum opened a new pull request #5621: [IOS] Fix build error of iOS RPC

kazum opened a new pull request #5621: URL: https://github.com/apache/incubator-tvm/pull/5621 This PR fixes build error of the iOS RPC. - Update to C++14 - Use the latest RPC protocol - Resolve CoreML dependency @tqchen @ZihengJiang @FrozenGene Please help to review. --

[GitHub] [incubator-tvm] littlefish0123 opened a new pull request #5620: Fix three typos

littlefish0123 opened a new pull request #5620: URL: https://github.com/apache/incubator-tvm/pull/5620 Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request co

[GitHub] [incubator-tvm] comaniac commented on pull request #5613: add schedule algorithm for super large batch and small inner dimension

comaniac commented on pull request #5613: URL: https://github.com/apache/incubator-tvm/pull/5613#issuecomment-630521369 > @comaniac I think the `dense_large_batch` is quite complicated, therefore adjusting the `dense_large_batch` directly to handle super large batch size is error-prone. In

[GitHub] [incubator-tvm] lsy643 commented on pull request #5613: add schedule algorithm for super large batch and small inner dimension

lsy643 commented on pull request #5613: URL: https://github.com/apache/incubator-tvm/pull/5613#issuecomment-630518642 @comaniac I think the `dense_large_batch` is quite complicated, therefore adjusting the `dense_large_batch` directly to handle super large batch size is error-prone. In ord

[GitHub] [incubator-tvm] kaimast closed issue #5608: Failed to compile Rust examples on SGX

kaimast closed issue #5608: URL: https://github.com/apache/incubator-tvm/issues/5608 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

[GitHub] [incubator-tvm] kaimast commented on issue #5608: Failed to compile Rust examples on SGX

kaimast commented on issue #5608: URL: https://github.com/apache/incubator-tvm/issues/5608#issuecomment-630485374 > https://github.com/apache/incubator-tvm/blob/master/apps/sgx/build.rs#L68-L73 Thanks. Not sure why but that fixes it!

[GitHub] [incubator-tvm] nhynes commented on issue #5608: Failed to compile Rust examples on SGX

nhynes commented on issue #5608: URL: https://github.com/apache/incubator-tvm/issues/5608#issuecomment-630475260 https://github.com/apache/incubator-tvm/blob/master/apps/sgx/build.rs#L68-L73 This is an automated message from

[GitHub] [incubator-tvm] mbrookhart opened a new pull request #5619: Add Scatter to Topi/Relay/ONNX via hybrid script

mbrookhart opened a new pull request #5619: URL: https://github.com/apache/incubator-tvm/pull/5619 cc: @jwfromm @tqchen @masahi The actual hybrid script implementation is kind of gross, I can't seem to figure out a way to clean it up without causing very opaque errors in the hybrid

[GitHub] [incubator-tvm] tqchen commented on pull request #5578: [Relay][Refactor][std::string --> String] Relay updated with String

tqchen commented on pull request #5578: URL: https://github.com/apache/incubator-tvm/pull/5578#issuecomment-630462825 Thanks @ANSHUMAN87 @jroesch @zhiics This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-tvm] tqchen merged pull request #5578: [Relay][Refactor][std::string --> String] Relay updated with String

tqchen merged pull request #5578: URL: https://github.com/apache/incubator-tvm/pull/5578 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 g

[incubator-tvm] branch master updated: [Relay][Refactor][std::string --> String] Relay updated with String (#5578)

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 538e296 [Relay][Refactor][std::string --

[GitHub] [incubator-tvm] jroesch commented on pull request #5596: [TIR][REFACTOR] std::string -> String Migration in TIR nodes

jroesch commented on pull request #5596: URL: https://github.com/apache/incubator-tvm/pull/5596#issuecomment-630390679 Let me know when its ready for review. This is an automated message from the Apache Git Service. To respon

[incubator-tvm] branch master updated (3be9576 -> 09792ec)

This is an automated email from the ASF dual-hosted git repository. jroesch pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 3be9576 [BYOC] Remove kCompiler attr from external functions (#5615) add 09792ec [Relay]Improve Shape

[GitHub] [incubator-tvm] jroesch merged pull request #5467: [Relay]Improve Shape Func handling for Tuple inputs

jroesch merged pull request #5467: URL: https://github.com/apache/incubator-tvm/pull/5467 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

[GitHub] [incubator-tvm] yzhliu opened a new pull request #5618: [Arith] Inequalities solver

yzhliu opened a new pull request #5618: URL: https://github.com/apache/incubator-tvm/pull/5618 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #5588: [Frontend][Tensorflow] Gather nd bug fix for one dim support in tensorflow

ANSHUMAN87 commented on pull request #5588: URL: https://github.com/apache/incubator-tvm/pull/5588#issuecomment-630361991 @kevinthesun : Your comment is handled now. Please check. Thanks! This is an automated message from the

[GitHub] [incubator-tvm] comaniac commented on a change in pull request #5616: [RELAY][BYOC] Fix the creation of tuple of tuples in PartitionGraph

comaniac commented on a change in pull request #5616: URL: https://github.com/apache/incubator-tvm/pull/5616#discussion_r426802467 ## File path: src/relay/transforms/partition_graph.cc ## @@ -404,18 +404,85 @@ IRModule RemoveDefaultAnnotations(IRModule module) { return modul

[GitHub] [incubator-tvm] zhiics commented on pull request #5615: [BYOC] Remove kCompiler attr from external functions

zhiics commented on pull request #5615: URL: https://github.com/apache/incubator-tvm/pull/5615#issuecomment-630338767 Thanks @lhutton1 @comaniac This is an automated message from the Apache Git Service. To respond to the mes

[incubator-tvm] branch master updated (2594f17 -> 3be9576)

This is an automated email from the ASF dual-hosted git repository. zhic pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from 2594f17 fix pattern topological order (#5612) add 3be9576 [BYOC] Remove kCompiler attr from external func

[GitHub] [incubator-tvm] zhiics merged pull request #5615: [BYOC] Remove kCompiler attr from external functions

zhiics merged pull request #5615: URL: https://github.com/apache/incubator-tvm/pull/5615 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 g

[GitHub] [incubator-tvm] deepakbabel23 commented on pull request #5617: [TENSORFLOW]StatefulPartitionedCall/PartitionedCall Ops support added

deepakbabel23 commented on pull request #5617: URL: https://github.com/apache/incubator-tvm/pull/5617#issuecomment-630334497 @kevinthesun , @yongwww Hi Kevin, Please review these changes. This is an automated message

[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #5578: [Relay][Refactor][std::string --> String] Relay updated with String

ANSHUMAN87 commented on pull request #5578: URL: https://github.com/apache/incubator-tvm/pull/5578#issuecomment-630326224 @tqchen : Your comment is handled now, added test case as well. Please check. Thanks! This is an autom

[GitHub] [incubator-tvm] kevinthesun commented on pull request #4312: [TOPI][Relay][OP] Dynamic NMS and strided_slice

kevinthesun commented on pull request #4312: URL: https://github.com/apache/incubator-tvm/pull/4312#issuecomment-630322373 @heliqi The code should be OK. This is an automated message from the Apache Git Service. To respond t

[GitHub] [incubator-tvm] tqchen commented on pull request #5612: Fix Topological Order calculation for DFPattern Language

tqchen commented on pull request #5612: URL: https://github.com/apache/incubator-tvm/pull/5612#issuecomment-630320123 THanks@mbrookhart @comaniac This is an automated message from the Apache Git Service. To respond to the

[incubator-tvm] branch master updated (f1fe9b0 -> 2594f17)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from f1fe9b0 Fix a typo. (#5611) add 2594f17 fix pattern topological order (#5612) No new revisions were ad

[GitHub] [incubator-tvm] tqchen merged pull request #5612: Fix Topological Order calculation for DFPattern Language

tqchen merged pull request #5612: URL: https://github.com/apache/incubator-tvm/pull/5612 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 g

[GitHub] [incubator-tvm] comaniac commented on pull request #5613: add schedule algorithm for super large batch and small inner dimension

comaniac commented on pull request #5613: URL: https://github.com/apache/incubator-tvm/pull/5613#issuecomment-630306466 Is there any obstacles that prevent you from improving `dense_large_batch` to support the batch size larger than 65535? -

[GitHub] [incubator-tvm] comaniac commented on issue #5605: [BUG] Pattern Reeriter Seg Fault

comaniac commented on issue #5605: URL: https://github.com/apache/incubator-tvm/issues/5605#issuecomment-630303281 The PR fixed this issue. Thanks! This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [incubator-tvm] comaniac closed issue #5605: [BUG] Pattern Reeriter Seg Fault

comaniac closed issue #5605: URL: https://github.com/apache/incubator-tvm/issues/5605 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 t

[GitHub] [incubator-tvm] deepakbabel23 opened a new pull request #5617: [TENSORFLOW]StatefulPartitionedCall/PartitionedCall Ops support added

deepakbabel23 opened a new pull request #5617: URL: https://github.com/apache/incubator-tvm/pull/5617 Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request cod

[GitHub] [incubator-tvm] tqchen commented on pull request #5600: [TOPI] Improve CUDA softmax scheduling

tqchen commented on pull request #5600: URL: https://github.com/apache/incubator-tvm/pull/5600#issuecomment-630282939 @icemelon9 @Hzfengsy @Shawn-Inspur please also take a look This is an automated message from the Apache Gi

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #5600: [TOPI] Improve CUDA softmax scheduling

tqchen commented on a change in pull request #5600: URL: https://github.com/apache/incubator-tvm/pull/5600#discussion_r426737561 ## File path: src/target/llvm/codegen_llvm.cc ## @@ -736,7 +736,40 @@ llvm::Function* CodeGenLLVM::GetIntrinsicDecl(llvm::Intrinsic::ID id, llvm::Ty

[GitHub] [incubator-tvm] wpan11nv commented on a change in pull request #5600: [TOPI] Improve CUDA softmax scheduling

wpan11nv commented on a change in pull request #5600: URL: https://github.com/apache/incubator-tvm/pull/5600#discussion_r426735612 ## File path: src/tir/transforms/lower_warp_memory.cc ## @@ -213,9 +213,13 @@ class WarpAccessRewriter : protected StmtExprMutator { alloc_siz

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #5578: [Relay][Refactor][std::string --> String] Relay updated with String

tqchen commented on a change in pull request #5578: URL: https://github.com/apache/incubator-tvm/pull/5578#discussion_r426677508 ## File path: include/tvm/relay/base.h ## @@ -93,7 +93,7 @@ class IdNode : public Object { * this only acts as a hint to the user, * and is

[incubator-tvm] branch master updated (cb7bd98 -> f1fe9b0)

This is an automated email from the ASF dual-hosted git repository. tqchen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-tvm.git. from cb7bd98 [CUDA] Fix codegen for warp shuffle intrinsics (#5606) add f1fe9b0 Fix a typo. (#5611) No new

[GitHub] [incubator-tvm] tqchen commented on pull request #5611: Fix a typo.

tqchen commented on pull request #5611: URL: https://github.com/apache/incubator-tvm/pull/5611#issuecomment-630221296 Thanks @littlefish0123 @cchung100m This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-tvm] tqchen merged pull request #5611: Fix a typo.

tqchen merged pull request #5611: URL: https://github.com/apache/incubator-tvm/pull/5611 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 g

[GitHub] [incubator-tvm] manupa-arm opened a new pull request #5616: [RELAY][BYOC] Fix the creation of tuple of tuples in PartitionGraph

manupa-arm opened a new pull request #5616: URL: https://github.com/apache/incubator-tvm/pull/5616 If the annotated compiler region contains multiple outputs where some of the outputs are tuple output, the current PartitionGraph will create tuple of tuples. This will not be handled by the

[GitHub] [incubator-tvm] lhutton1 commented on pull request #5615: [BYOC] Remove kCompiler attr from external functions

lhutton1 commented on pull request #5615: URL: https://github.com/apache/incubator-tvm/pull/5615#issuecomment-630216977 cc @comaniac @zhiics @mbaret This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-tvm] lhutton1 opened a new pull request #5615: [BYOC] Remove kCompiler attr from external functions

lhutton1 opened a new pull request #5615: URL: https://github.com/apache/incubator-tvm/pull/5615 Functions destined for external codegen keep their kCompiler attribute which means SkipFunction returns true when running a pass over such functions during the codegen step. This makes sense du

[GitHub] [incubator-tvm] siju-samuel opened a new pull request #5614: [MXNET]MaxPool3d and AvgPool3d Ops support added

siju-samuel opened a new pull request #5614: URL: https://github.com/apache/incubator-tvm/pull/5614 MaxPool3D and AvgPool3D op support added in mxnet. @FrozenGene @masahi Please help to review this PR. TIA This is

[GitHub] [incubator-tvm] siju-samuel commented on a change in pull request #5394: [TFLITE]Quantize & Dequantize op

siju-samuel commented on a change in pull request #5394: URL: https://github.com/apache/incubator-tvm/pull/5394#discussion_r426560071 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1552,6 +1552,48 @@ def test_forward_squeeze(): _test_squeeze(np.arange(6)

[GitHub] [incubator-tvm] siju-samuel commented on a change in pull request #5394: [TFLITE]Quantize & Dequantize op

siju-samuel commented on a change in pull request #5394: URL: https://github.com/apache/incubator-tvm/pull/5394#discussion_r426561425 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1552,6 +1552,48 @@ def test_forward_squeeze(): _test_squeeze(np.arange(6)

[GitHub] [incubator-tvm] siju-samuel commented on a change in pull request #5394: [TFLITE]Quantize & Dequantize op

siju-samuel commented on a change in pull request #5394: URL: https://github.com/apache/incubator-tvm/pull/5394#discussion_r426560071 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1552,6 +1552,48 @@ def test_forward_squeeze(): _test_squeeze(np.arange(6)

[GitHub] [incubator-tvm] inadob commented on a change in pull request #5394: [TFLITE]Quantize & Dequantize op

inadob commented on a change in pull request #5394: URL: https://github.com/apache/incubator-tvm/pull/5394#discussion_r426555314 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1552,6 +1552,48 @@ def test_forward_squeeze(): _test_squeeze(np.arange(6).resh

[GitHub] [incubator-tvm] FrozenGene commented on a change in pull request #5394: [TFLITE]Quantize & Dequantize op

FrozenGene commented on a change in pull request #5394: URL: https://github.com/apache/incubator-tvm/pull/5394#discussion_r426550701 ## File path: tests/python/frontend/tflite/test_forward.py ## @@ -1552,6 +1552,48 @@ def test_forward_squeeze(): _test_squeeze(np.arange(6).

[GitHub] [incubator-tvm] siju-samuel commented on pull request #5394: [TFLITE]Quantize & Dequantize op

siju-samuel commented on pull request #5394: URL: https://github.com/apache/incubator-tvm/pull/5394#issuecomment-630102640 @masahi @anijain2305 @inadob Please help to review and merge this PR. TIA This is an automated message

[GitHub] [incubator-tvm] siju-samuel commented on pull request #5098: [KERAS]Global MaxPool3d and AvgPool3d support

siju-samuel commented on pull request #5098: URL: https://github.com/apache/incubator-tvm/pull/5098#issuecomment-630102255 @masahi Please help to review and merge. TIA. This is an automated message from the Apache Git Ser

[GitHub] [incubator-tvm] heliqi commented on pull request #4312: [TOPI][Relay][OP] Dynamic NMS and strided_slice

heliqi commented on pull request #4312: URL: https://github.com/apache/incubator-tvm/pull/4312#issuecomment-630033704 Is there anything wrong with this code? Can I use it directly now? My model has an operator "NonMaxSuppressionv4", only one more parameter "pad_to_max_output_size". I

[GitHub] [incubator-tvm] Sophia0806 commented on issue #4490: Issue with running cpp test

Sophia0806 commented on issue #4490: URL: https://github.com/apache/incubator-tvm/issues/4490#issuecomment-630008095 Hi have you resolved this problem? I meet the same one. This is an automated message from the Apache Git Ser