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.151】PaddleRS运行环境打包,并制作端到端遥感建筑物提取教程 #147

Merged
merged 25 commits into from
May 10, 2023

Conversation

geoyee
Copy link
Collaborator

@geoyee geoyee commented May 9, 2023

打包完成,测试paddlers、eiseg和geoview可用,docker 19以上可使用gpu,但最后geoview推理上有一点问题,可能与paddlers的更新有关,已反馈,等待修复。

Copy link
Member

@Bobholamovic Bobholamovic left a comment

Choose a reason for hiding this comment

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

想讨论一下:这个教程是否放在examples中更合适呀?考虑到EISeg对于PaddleRS来说是友情项目,而GeoView目前也是定位为一个基于PaddleRS开发的『官方案例』,它们都不是PaddleRS本体的一部分。

.gitignore Outdated Show resolved Hide resolved
docker/Dockerfile.geoview Outdated Show resolved Hide resolved
docker/Dockerfile.geoview Outdated Show resolved Hide resolved
docker/Dockerfile.geoview Outdated Show resolved Hide resolved
docker/Dockerfile.geoview Outdated Show resolved Hide resolved
Copy link
Member

@Bobholamovic Bobholamovic left a comment

Choose a reason for hiding this comment

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

留了一些意见,请考虑一下~

docker/Dockerfile.geoview Outdated Show resolved Hide resolved
examples/docker_workflow/README.md Outdated Show resolved Hide resolved
examples/docker_workflow/README.md Outdated Show resolved Hide resolved

- 上传图像,开始处理,就能得到可视化的结果了。

*\*由于PaddleRS经过了一段时间的更新,目前GeoView推理时暂时有点问题,结果待更正*
Copy link
Member

Choose a reason for hiding this comment

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

GeoView作为PaddleRS的下游应用,需要依赖特定版本的PaddleRS,在下一个稳定版本前应该暂时不会支持最新版的PaddleRS~ 教程中可以引导用户在GeoView以submodule形式集成的frozen PaddleRS中训练导出模型。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

不过不是新的paddlers快出了吗,那边大佬也说会调整一下,感觉到时候发版的时候在dockerfile里面把paddlers和geoview的clone之后都checkout到release版本应该就ok了,就不会有版本问题了。晖哥你觉得呢

docker/Dockerfile.geoview Outdated Show resolved Hide resolved
docker/README_EN.md Outdated Show resolved Hide resolved
Copy link
Member

@Bobholamovic Bobholamovic left a comment

Choose a reason for hiding this comment

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

LGTM.

@Bobholamovic Bobholamovic merged commit 389330b into PaddlePaddle:develop May 10, 2023
4 of 6 checks passed
@geoyee geoyee deleted the docker branch May 11, 2023 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants