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 for multiple bugs in evolve operation mentioned in issue 1611 #1698

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nahidalam
Copy link

Currently there are multiple bugs in --evolve operation in the yolov7 repository.

  • Bug 1: Evolve operation fails if there is zero weight in yolov7/train.py, line 674
x = x[random.choices(range(n), weights=w)[0]] # weighted selection

Solution of this bug is implemented here ultralytics/yolov5@e095b74

  • Bug 2: Even if the bug 1 above is solved, users encountered below issue in yolov7/train.py line 688
hyp[k] = float(x[i + 7] * v[i])  # mutate
IndexError: index 30 is out of bounds for axis 0 with size 30

This is because the meta dictionary is not including the loss_ota parameter which is a new hyperparameter in yolov7.

Both the bugs are reported by multiple users in the issue here #1611

This PR resolves both of those bugs.

@shossain
Copy link

ship it!

Copy link

@lnhutnam lnhutnam left a comment

Choose a reason for hiding this comment

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

Tks u. It's very helpful :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants