Skip to content

Fix remove squeeze#571

Merged
gttiankai merged 10 commits into
Tencent:masterfrom
1627180283:fix_remove_squeeze
Dec 2, 2020
Merged

Fix remove squeeze#571
gttiankai merged 10 commits into
Tencent:masterfrom
1627180283:fix_remove_squeeze

Conversation

@1627180283
Copy link
Copy Markdown
Collaborator

Fix the bug caused by squeeze node before the output node.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 20, 2020

Codecov Report

Merging #571 (64f482f) into master (e4710bf) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #571   +/-   ##
=======================================
  Coverage   31.02%   31.02%           
=======================================
  Files         290      290           
  Lines        9280     9280           
=======================================
  Hits         2879     2879           
  Misses       6401     6401           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4710bf...64f482f. Read the comment docs.

const std::string& node_output_name = node->output(0);
if (output_node_set.find(node_output_name) != output_node_set.end() && i > 0) {
const auto& node_input_name = node->input(0);
auto pre_node = index_nodes[i - 1].node;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

编号为 i-1 的layer 一定是 squeeze 的前一个算子吗?

@gttiankai gttiankai merged commit 9a58baf into Tencent:master Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants