-
Notifications
You must be signed in to change notification settings - Fork 92
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
Conversation
There was a problem hiding this 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本体的一部分。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
留了一些意见,请考虑一下~
examples/docker_workflow/README.md
Outdated
|
||
- 上传图像,开始处理,就能得到可视化的结果了。 | ||
|
||
*\*由于PaddleRS经过了一段时间的更新,目前GeoView推理时暂时有点问题,结果待更正* |
There was a problem hiding this comment.
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中训练导出模型。
There was a problem hiding this comment.
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了,就不会有版本问题了。晖哥你觉得呢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
打包完成,测试paddlers、eiseg和geoview可用,docker 19以上可使用gpu,但最后geoview推理上有一点问题,可能与paddlers的更新有关,已反馈,等待修复。