soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586529239
This test (test_branch_merge) is now failing at the de-duplicate pass. It's
interesting since the
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586529239
This test (test_branch_merge) is now failing at the de-duplicate pass. It's
interesting since the
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586529239
This test (test_branch_merge) is now failing at the de-duplicate pass. It's
interesting since the
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586529239
This test (test_branch_merge) is now failing at the de-duplicate pass. It's
interesting since the
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586529239
This test is now failing at the de-duplicate pass. It's interesting since
the "correct" (result)
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586527376
This is actually a good exercise, as some other tests are actually failing
now due to the graphs
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586527376
This is actually a good exercise, as some other tests are actually failing
now due to the graphs
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586521847
Actually, are you sure the second argument is expected? It looks like
AlphaEqual loops through the
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586472422
I'll do some investigation :)
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586472422
I'll do some investigation :)
Update: in alpha_equal.cc, `relay._make._graph_equal` actually
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586472422
I'll do some investigation :)
Update: in alpha_equal.cc, `relay._make._graph_equal` actually
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586428127
Yes, they pass the alpha_equal check.
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586412142
@mbarrett97 @comaniac I just pushed a test where "result" creates an
incorrect graph, and
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586412142
@mbarrett97 @comaniac I just pushed a test where "result" creates an
incorrect graph, and
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586412142
@mbarrett97 @comaniac I just pushed a test where "result" creates an
incorrect graph, and
soiferj edited a comment on issue #4879: [Relay][Pass] Fix bug in re-processing
call node in MergeComposite pass
URL: https://github.com/apache/incubator-tvm/pull/4879#issuecomment-586412142
@mbarrett97 @comaniac I just pushed a test where "result" creates an
incorrect graph, and
16 matches
Mail list logo
