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.

Sign up for GitHub

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

Jump to bottom

minor fix on max_iters parameter passing bug #464

Merged

DavdGao merged 1 commit into modelscope:main from zzzmc:main

Oct 24, 2024

Contributor

zzzmc commented Oct 17, 2024 •

edited

Loading

name: Pull Request
about: Create a pull request

Description

Fixed a minor bug where the "max_iters" parameter was not used in the for loop.

Checklist

Please check the following items before code is ready to be reviewed.

Code has passed all tests
Docstrings have been added/updated in Google Style
Documentation has been updated
Code is ready for review


          minor fix on max_iters parameter passing bug

8903a21

DavdGao approved these changes

View reviewed changes

Collaborator

DavdGao left a comment

LGTM

DavdGao merged commit eeee97a into modelscope:main

13 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet