-
Notifications
You must be signed in to change notification settings - Fork 37
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
[Feature] 自定义权重的前端适配 #33
Merged
Bobholamovic
merged 28 commits into
PaddleCV-SIG:develop
from
yibaikuai:feature/custom-weight
Oct 13, 2022
Merged
[Feature] 自定义权重的前端适配 #33
Bobholamovic
merged 28 commits into
PaddleCV-SIG:develop
from
yibaikuai:feature/custom-weight
Oct 13, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
yibaikuai
commented
Oct 12, 2022
•
edited
Loading
edited
- 实现自定义权重的前端适配
- 解决缩放浏览器时变化检测功能区两个”上传文件夹“布局混乱的问题
- 修复变化检测功能区清空图片按钮的布局bug
- 修改文件名的命名
# Conflicts: # backend/applications/interface/object_detection.py
* 新增场景分类功能 * 1.修复绘制目标框不准确的bug 2.修复图像偏色问题 3.将目标框颜色改为更显眼的紫色 * 修改滑窗步长,加快推理速度 * 1.修改场景分类读取图像的接口,改用paddelrs中的decode_image 2.修改地物分类结果图像的颜色显示 3.去除地物分类任务中的冗余代码 * 删除地物分类中冗余代码 * 1.修改错误函数名(批量处理) 2.删除渲染函数中冗余代码 3.改用cv2统一对图像进行操作处理,去除skimage的引用 * 1.变化检测滑窗推理部分改为PaddleRS原生的slider_predict()方法 2.滑窗尺寸和步长改为可调的 3.修改目标检测代码中的不恰当注释 4.修改错误函数名(上次没看到,梅开二度) * 新增图像修复功能 * lint
修复type为空的bug
jscslld
requested changes
Oct 13, 2022
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,但在测试过程中发现了图片下载压缩功能逻辑不正确的历史遗留问题,已向作者反馈进行反馈修改,等修复后一起合入。此外测试中发现了后端存在目录不存在时抛出未预料的异常的问题,已通过 #34 进行修复。
jscslld
approved these changes
Oct 13, 2022
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.