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

Alexnet TinyMs XJY #59

Merged
merged 16 commits into from
Apr 1, 2021
Merged

Alexnet TinyMs XJY #59

merged 16 commits into from
Apr 1, 2021

Conversation

JingyangXiang
Copy link
Contributor

加入了alexnet 运行请见alexnet.md

@CLAassistant
Copy link

CLAassistant commented Apr 1, 2021 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@JingyangXiang JingyangXiang changed the title Alexnet Alexnet TinyMs XJY Apr 1, 2021
leonwanghui
leonwanghui reviewed Apr 1, 2021
View reviewed changes
tests/st/alexnet.py Outdated Show resolved Hide resolved
@leonwanghui
Copy link
Contributor

@zjuter0126 Some tiny errors mentioned above

@JingyangXiang
Copy link
Contributor Author

@zjuter0126 Some tiny errors mentioned above

I have corrected my errors mentioned above.

@leonwanghui
Copy link
Contributor

@zjuter0126 Got it, I'm testing your PR locally to see if they are working well, please hold a minute : )

leonwanghui
leonwanghui previously approved these changes Apr 1, 2021
View reviewed changes
Copy link
Contributor

@leonwanghui leonwanghui left a comment

Choose a reason for hiding this comment

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

LGTM

@huxiaoman7
Copy link
Collaborator

LGTM

huxiaoman7
huxiaoman7 previously approved these changes Apr 1, 2021
View reviewed changes
@JingyangXiang
Copy link
Contributor Author

LGTM
Thank you, the code for densenet is similar to alexnet, you can run it in a similar way.

@leonwanghui
Copy link
Contributor

@zjuter0126 Please remove the *.md and *.png file under the root folder, then this PR would be getting merged.

@JingyangXiang JingyangXiang dismissed stale reviews from huxiaoman7 and leonwanghui via 963ce5b April 1, 2021 09:33
@JingyangXiang
Copy link
Contributor Author

@zjuter0126 Please remove the *.md and *.png file under the root folder, then this PR would be getting merged.

OK，I have remove the *.md and *.png

leonwanghui
leonwanghui approved these changes Apr 1, 2021
View reviewed changes
Copy link
Contributor

@leonwanghui leonwanghui left a comment

Choose a reason for hiding this comment

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

LGTM

@huxiaoman7
Copy link
Collaborator

LGTM

@leonwanghui leonwanghui merged commit d5aaaac into tinyms-ai:main Apr 1, 2021
@JingyangXiang JingyangXiang deleted the alexnet branch May 24, 2021 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leonwanghui leonwanghui leonwanghui approved these changes

@huxiaoman7 huxiaoman7 huxiaoman7 approved these changes

Assignees

@JingyangXiang JingyangXiang

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JingyangXiang @CLAassistant @leonwanghui @huxiaoman7