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

add test_create_loader #162

Merged
merged 1 commit into from
Jul 6, 2023
Merged

add test_create_loader #162

merged 1 commit into from
Jul 6, 2023

Conversation

yuedongli1
Copy link
Collaborator

@yuedongli1 yuedongli1 commented Jun 29, 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 inside-test 内部开发者提的issue rfc 需求单issue labels Jun 29, 2023
@yuedongli1 yuedongli1 added this to the mindyolo-0.1 milestone Jun 29, 2023
@yuedongli1 yuedongli1 self-assigned this Jun 29, 2023
@yuedongli1 yuedongli1 linked an issue Jun 29, 2023 that may be closed by this pull request
Comment on lines 37 to 38
wget https://github.com/ultralytics/yolov5/releases/download/v1.0/coco128.zip
unzip coco128.zip
Copy link
Collaborator

Choose a reason for hiding this comment

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

@@ -45,7 +45,7 @@ def test_create_trainer(yaml_name, mode):
network.set_train(True)

# Create Dataloaders
bs = 8
bs = 6
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个地方为什么修改为6

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

8有时候会超内存

@CaitinZhao CaitinZhao merged commit af76684 into mindspore-lab:master Jul 6, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inside-test 内部开发者提的issue rfc 需求单issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add unit test (UT)
3 participants