[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449998348 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,466 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449997959 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,470 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] junrushao1994 commented on pull request #5988: [TARGET] each option of target str should only contain one '='

junrushao1994 commented on pull request #5988: URL: https://github.com/apache/incubator-tvm/pull/5988#issuecomment-654030410 @windclarion Looks like a flaky test. Could you file an empty commit using `git commit --allow-empty -m "retrigger"` to retrigger the CI?

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449996272 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449996272 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449995770 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449995770 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449991342 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,153 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449990416 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,141 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] leonwanghui commented on pull request #5892: Add TVM application extension with WASM runtime

leonwanghui commented on pull request #5892: URL: https://github.com/apache/incubator-tvm/pull/5892#issuecomment-654001303 @tqchen @jroesch @kazum Now this PR is updated, please take a review, thanks! This is an automated

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449967045 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,141 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

junrushao1994 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449966709 ## File path: src/ansor/compute_dag.cc ## @@ -0,0 +1,505 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449966453 ## File path: python/tvm/ansor/auto_schedule.py ## @@ -0,0 +1,200 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449964790 ## File path: src/ansor/compute_dag.cc ## @@ -0,0 +1,505 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449964659 ## File path: src/ansor/compute_dag.cc ## @@ -0,0 +1,505 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449964322 ## File path: src/ansor/compute_dag.cc ## @@ -0,0 +1,505 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449955848 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] yangjing69 opened a new issue #5996: How to call DNNL under TVM version 0.7

yangjing69 opened a new issue #5996: URL: https://github.com/apache/incubator-tvm/issues/5996 I'm using tvm version 0.7 and need to call the DNNL library, but tried the following statements and found that none of them are supported . mod = relay.build_extern(mod, "dnnl") mod =

[GitHub] [incubator-tvm] lixiaoquan commented on pull request #5963: [TF]Refine LSTMBlockCell to support dynamic rnn

lixiaoquan commented on pull request #5963: URL: https://github.com/apache/incubator-tvm/pull/5963#issuecomment-653985373 also cc @zhiics @yongwww Could you please help to reivew? Thanks This is an automated message from

[GitHub] [incubator-tvm] masahi merged pull request #5995: [Relay][Frontend][Onnx] Small bug fix for Conv1D imports.

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

[incubator-tvm] branch master updated: [Relay][Frontend][Onnx] Small bug fix for Conv1D imports. (#5995)

This is an automated email from the ASF dual-hosted git repository. masahi 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 a64feed [Relay][Frontend][Onnx] Small

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449955848 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449955188 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449955188 ## File path: python/tvm/ansor/measure.py ## @@ -0,0 +1,386 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449953100 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,153 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449953100 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,153 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] merrymercy commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

merrymercy commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449953100 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,153 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] windclarion commented on pull request #5988: [TARGET] each option of target str should only contain one '='

windclarion commented on pull request #5988: URL: https://github.com/apache/incubator-tvm/pull/5988#issuecomment-653973990 done! old code works ok before "[Target] Migrate data structure of TargetNode (#5960)", the commit add a requirement in src/target/target_id.cc. FindUniqueSubstr

[GitHub] [incubator-tvm] yangjunpro commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

yangjunpro commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449937406 ## File path: python/tvm/ansor/workload_registry.py ## @@ -0,0 +1,170 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [incubator-tvm] junrushao1994 commented on pull request #5988: [TARGET] each option of target str should only contain one '='

junrushao1994 commented on pull request #5988: URL: https://github.com/apache/incubator-tvm/pull/5988#issuecomment-653957802 Ah I see, I am a bit surprised the typo is not found before. BTW, please rebase to master, and change “-target” to “-mtriple”. Thanks!

[GitHub] [incubator-tvm] junrushao1994 commented on pull request #5987: [TARGET] each option of target str should only contain one '='

junrushao1994 commented on pull request #5987: URL: https://github.com/apache/incubator-tvm/pull/5987#issuecomment-653957078 Hey could you elaborate the issue you encounter and provide a minimal example to reproduce? Thank you!

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

junrushao1994 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449933639 ## File path: src/ansor/loop_state.cc ## @@ -0,0 +1,447 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] junrushao1994 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

junrushao1994 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449933549 ## File path: src/ansor/loop_state.cc ## @@ -0,0 +1,447 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] tqchen commented on pull request #5993: [Target] Use TargetNode::attrs for Target serialization

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

[GitHub] [incubator-tvm] tqchen merged pull request #5993: [Target] Use TargetNode::attrs for Target serialization

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

[incubator-tvm] branch master updated (575a383 -> 67a7497)

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 575a383 [TFLite] QNN support for TFLite 2.1.0 quantized models (#5848) add 67a7497 [Target] Use

[GitHub] [incubator-tvm] tqchen commented on a change in pull request #5993: [Target] Use TargetNode::attrs for Target serialization

tqchen commented on a change in pull request #5993: URL: https://github.com/apache/incubator-tvm/pull/5993#discussion_r449897101 ## File path: include/tvm/target/target_id.h ## @@ -60,30 +60,35 @@ class TargetIdNode : public Object { int device_type; /*! \brief Default

[GitHub] [incubator-tvm] jwfromm commented on pull request #5995: [Relay][Frontend][Onnx] Small bug fix for Conv1D imports.

jwfromm commented on pull request #5995: URL: https://github.com/apache/incubator-tvm/pull/5995#issuecomment-653910795 @masahi, @areusch, @tmoreau89 can one of you guys take a look at this quick PR? This is an automated

[GitHub] [incubator-tvm] jwfromm opened a new pull request #5995: [Relay][Frontend][Onnx] Small bug fix for Conv1D imports.

jwfromm opened a new pull request #5995: URL: https://github.com/apache/incubator-tvm/pull/5995 When importing a Conv1D operation with VALID padding, a Relay error would be produced due to the onnx frontend producing a tuple with only one element instead of two as expected. Adding

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449890672 ## File path: src/ansor/loop_state.cc ## @@ -0,0 +1,447 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449890449 ## File path: src/ansor/compute_dag.cc ## @@ -0,0 +1,505 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r44989 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,153 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] yangjunpro commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

yangjunpro commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449876496 ## File path: python/tvm/ansor/compute_dag.py ## @@ -0,0 +1,153 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] yangjunpro commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

yangjunpro commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449864276 ## File path: python/tvm/ansor/auto_schedule.py ## @@ -0,0 +1,200 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] yangjunpro commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

yangjunpro commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449861666 ## File path: python/tvm/ansor/auto_schedule.py ## @@ -0,0 +1,206 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

[GitHub] [incubator-tvm] yangjunpro commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

yangjunpro commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449859957 ## File path: tests/python/unittest/test_ansor_search_policy.py ## @@ -0,0 +1,86 @@ +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449851109 ## File path: src/ansor/loop_state.cc ## @@ -0,0 +1,447 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449849918 ## File path: src/ansor/measure.cc ## @@ -0,0 +1,331 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

[GitHub] [incubator-tvm] jcf94 commented on a change in pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on a change in pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#discussion_r449848604 ## File path: src/ansor/loop_state.cc ## @@ -0,0 +1,447 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor

[GitHub] [incubator-tvm] jcf94 commented on pull request #5962: [Ansor][AutoTVM v2.0] Part 0: Ansor minimum system for auto schedule generating

jcf94 commented on pull request #5962: URL: https://github.com/apache/incubator-tvm/pull/5962#issuecomment-653847183 > Thank you for the contribution! Just did a round of review on the c++ side. Code is pretty well written and organized - really enjoy it when reading them through.

[GitHub] [incubator-tvm] ANSHUMAN87 commented on pull request #5932: [Frontend][Relay] Add Parser 2.0

ANSHUMAN87 commented on pull request #5932: URL: https://github.com/apache/incubator-tvm/pull/5932#issuecomment-653846659 @jroesch : Thanks for the PR! Great work :+1: I totally agree with the motivations behind this, you have mentioned. I am sorry i could not find any RFC for this