Skip to content

[AutoTVM] Fix Split Factors when no_tail is off - #4044

Merged
vinx13 merged 1 commit into
apache:masterfrom
comaniac:fix_no_tail
Oct 1, 2019
Merged

[AutoTVM] Fix Split Factors when no_tail is off#4044
vinx13 merged 1 commit into
apache:masterfrom
comaniac:fix_no_tail

Conversation

@comaniac

Copy link
Copy Markdown
Contributor
  • Fix the issue that define_split may generate a config that the product of all factors is larger than the axis length when no_tail is turned off.

  • Add unit tests.

@comaniac

Copy link
Copy Markdown
ContributorAuthor

@vinx13 could you help review this PR? Thanks.

@vinx13
vinx13 merged commit 2d53762 into apache:masterOct 1, 2019
@comaniac
comaniac deleted the fix_no_tail branch October 1, 2019 23:44
anijain2305 pushed a commit to anijain2305/tvm that referenced this pull request Oct 17, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Oct 18, 2019
petrex pushed a commit to petrex/tvm that referenced this pull request Oct 29, 2019
* master:
Fix split's last factor issue (apache#4044)
[COMMUNITY] ajtulloch -> committer (apache#4043)
[TOPI]Add op argwhere (apache#3994)
[topi] add ARM v8.2 udot (uint8) support (apache#3978)
[COMMUNITY] anijain2305 -> reviewer (apache#4036)
[QNN] Renaming dense operator. (apache#4033)
[Relay][Compile_engine] Int64 shape handling for outputs. (apache#4031)
Add dmlc-core to the list of installed header directories. (apache#4035)
[ARITH] migrate indexdiv/mod to floordiv/mod (apache#4008)
[Relay] Move prelude to text format (apache#3939)
make tvm compilable by gcc 4.9.2 (apache#4032)
[AUTOTVM][DOCS] Add a link to the defining network description of auto-tuning tutorial (apache#4023)
[ARITH] cleanup the indexmod/div on python side (apache#4028)
[Fix] Add more pad_mode support for onnx converter (apache#4029)
Add parser support for ReLU tflite operator (apache#4022)
Additional MXNet Convolution and Deconvolution tests (apache#4026)
docs: minor spelling tweaks (apache#4027)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@comaniac@vinx13