Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[New Feature] Support Paddle PIR Visualization 🚀 #1263

Merged
merged 3 commits into from
Sep 20, 2023

Conversation

Difers
Copy link
Contributor

@Difers Difers commented Sep 19, 2023

基于VisualDL为Paddle新IR添加可视化

相关链接

主要改动

  • 添加analyse_program对新IR program可视化
  • 添加program_test测例

其它改动

  • pre-commit中flake8仓库从gitlab改为github,便于使用
  • app.py中 lang= request.accept_languages.best_match(support_language)可能为None导致visualDL启动失败,故改动

运行测例

  1. export FLAGS_enable_new_ir_in_executor=true
  2. python demo/components/program_test.py 输出文件将在xxx/VisualDL/log/program_test下
  3. visualdl --logdir ~/VisualDL/log/program_test/ --port 8080 注意需绝对路径

当前效果

image

Copy link
Collaborator

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

Great work!

.pre-commit-config.yaml Show resolved Hide resolved
demo/components/program_test.py Outdated Show resolved Hide resolved
visualdl/component/graph/exporter.py Outdated Show resolved Hide resolved
visualdl/component/graph/graph_component.py Outdated Show resolved Hide resolved
visualdl/component/graph/utils.py Show resolved Hide resolved
visualdl/writer/writer.py Outdated Show resolved Hide resolved
visualdl/writer/writer.py Outdated Show resolved Hide resolved
Aurelius84
Aurelius84 previously approved these changes Sep 19, 2023
Copy link
Collaborator

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

LGTM overall

visualdl/writer/writer.py Outdated Show resolved Hide resolved
demo/components/program_test.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

LGTM

@Aurelius84 Aurelius84 changed the title Paddle新IR可视化 [New Feature] Support Paddle PIR Visualization Sep 20, 2023
@Aurelius84 Aurelius84 changed the title [New Feature] Support Paddle PIR Visualization [New Feature] Support Paddle PIR Visualization 🚀 Sep 20, 2023
Copy link
Collaborator

@rainyfly rainyfly left a comment

Choose a reason for hiding this comment

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

LGTM

@rainyfly rainyfly merged commit 6c9757b into PaddlePaddle:develop Sep 20, 2023
3 of 4 checks passed
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