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.

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

Fix a bug in the validation script of the conformer #189

Merged
merged 1 commit into from
Jul 11, 2024
Merged

Fix a bug in the validation script of the conformer #189

merged 1 commit into from
Jul 11, 2024
+3 −1

Conversation

PingqiLi
Copy link
Collaborator

What does this PR do?

  • Fix a bug in the validation script of the conformer (examples/conformer/predict.py)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline?
  • Did you build and run the code without any errors?
  • Did you report the running environment (NPU type/MS version) and performance in the doc? (better record it for data loading, model inference, or training tasks)
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@Ash-Lee233 Ash-Lee233 merged commit 8c8cc12 into mindspore-lab:main Jul 11, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LiTingyu1997 LiTingyu1997 LiTingyu1997 approved these changes

@Ash-Lee233 Ash-Lee233 Ash-Lee233 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PingqiLi @Ash-Lee233 @LiTingyu1997