Skip to content

remove graph visualizer tool, which can be replaced by python IrGraph draw API#16177

Merged
luotao1 merged 1 commit intoPaddlePaddle:developfrom
fc500110:remove_visualizer
Mar 21, 2019
Merged

remove graph visualizer tool, which can be replaced by python IrGraph draw API#16177
luotao1 merged 1 commit intoPaddlePaddle:developfrom
fc500110:remove_visualizer

Conversation

@fc500110
Copy link
Contributor

No description provided.

@fc500110 fc500110 changed the title remove graph visualizer, which can be replaced by python IrGraph draw API remove graph visualizer tool, which can be replaced by python IrGraph draw API Mar 14, 2019
@fc500110 fc500110 requested a review from panyx0718 March 15, 2019 02:56
@wzzju wzzju self-requested a review March 18, 2019 07:26
Copy link
Contributor

@wzzju wzzju left a comment

Choose a reason for hiding this comment

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

The visualizer tool should be deleted, because this function has been included in IrGraph. What's more, the visualizer tool is very huge, and is about 1.4GB. What a mess!

@luotao1 luotao1 merged commit 294cdf6 into PaddlePaddle:develop Mar 21, 2019
@fc500110 fc500110 deleted the remove_visualizer branch March 21, 2019 12:48
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