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

fix yolox yaml #255

Merged
merged 1 commit into from
Dec 18, 2023
Merged

fix yolox yaml #255

merged 1 commit into from
Dec 18, 2023

Conversation

yuedongli1
Copy link
Collaborator

@yuedongli1 yuedongli1 commented Dec 11, 2023

Thank you for your contribution to the MindYOLO repo.
Before submitting this PR, please make sure:

Motivation

(Write your motivation for proposed changes here.)

Test Plan

(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)

Related Issues and PRs

(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

@yuedongli1 yuedongli1 added bug Something isn't working inside-test 内部开发者提的issue labels Dec 11, 2023
@yuedongli1 yuedongli1 added this to the mindyolo-0.1 milestone Dec 11, 2023
@yuedongli1 yuedongli1 self-assigned this Dec 11, 2023
@yuedongli1 yuedongli1 linked an issue Dec 11, 2023 that may be closed by this pull request
zhanghuiyao
zhanghuiyao previously approved these changes Dec 14, 2023
@@ -56,7 +56,6 @@ data:
],
[
{ func_name: letterbox, scaleup: False },
{ func_name: resample_segments },
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个删掉会影响其他yolox吗

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

不会的,这个就是上次改数据处理的时候为了不报错加上来的,现在修改了fliplr中进入判断segments格式的条件,应该是更好的方案了

@CaitinZhao CaitinZhao merged commit 867bc31 into mindspore-lab:master Dec 18, 2023
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working inside-test 内部开发者提的issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

yolox-nano/tiny的yaml有问题
3 participants