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

【Hackathon No.166】Paddle3D目标检测结果可视化 #256

Closed
wants to merge 8 commits into from

Conversation

lizechng
Copy link

@lizechng lizechng commented Feb 27, 2023

PR types

Others

PR changes

Others

Describe

完成单目图像/鸟瞰图/点云可视化
bev
pc

output

return Scheduler(**kwargs)


class Infer:
Copy link
Collaborator

Choose a reason for hiding this comment

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

很多内容和trainer重复了,可以继承trainer来实现,同时该文件需要放到paddle3d里

Copy link
Author

Choose a reason for hiding this comment

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

您好,我已经进行修改重新提交PR,链接: #272

@LielinJiang
Copy link
Collaborator

需要给develop提PR哈,release是稳定的发布版本

@lizechng
Copy link
Author

需要给develop提PR哈,release是稳定的发布版本

已经向develop重新提交PR,CI已经通过,麻烦您再看一下

@thinkthinking
Copy link
Collaborator

你好,有以下几个修改建议:

  1. 目录结构&命名规范希望按照如下图所示修改一下:
image
  1. readme里面补充一下几个不同可视化示例的使用说明以及命令行参数的说明;
  2. readme里面补充一下可视化api的使用说明;
  3. readme文档同步提供中文&英文文档,置于父目录docs/visualization下面,文档名称改为visualization_en.md, visualization_cn.md
  4. readme文档开头的安装链接到Paddle3D的标准安装教程https://github.com/PaddlePaddle/Paddle3D/blob/develop/docs/installation.md
  5. 可视化所需的额外安装包同步提PR到Paddle3D的requirements.txt里面

@lizechng lizechng closed this May 30, 2023
@paddle-bot paddle-bot bot added the contributor External Developers label Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External Developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants