Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Refine this repo #11

Merged
merged 9 commits into from
Oct 12, 2017
Merged

Refine this repo #11

merged 9 commits into from
Oct 12, 2017

Conversation

hedaoyuan
Copy link
Collaborator

No description provided.

@hedaoyuan hedaoyuan requested a review from NHZlX October 11, 2017 05:35

## Model compression
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand why separate the rounding and pruning

Copy link
Contributor

Choose a reason for hiding this comment

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

how about set Deployment optimization methods to Tools and move the rounding to Model compression

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

As with the merge batch normalization, the rounding method is used to process the model before the inference. So I put it in the tool directory.


| Model | Xiaomi MI 5(ST) |
|-----------| ----------- |
| Mobilenet | 122.607(ms) |
Copy link
Contributor

Choose a reason for hiding this comment

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

need to specify the type of mobilenet (1.0 or 0.75 , 0.5 0.25) and input size(224* 224 * 3)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

name="output", input=net, size=classdim, act=paddle.activation.Softmax())

Inputs("image")
Outputs("output")
Copy link
Contributor

Choose a reason for hiding this comment

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

This config can not be used in v2 api, all mobilenet config in this repo should be the same specifications, and could use for v1 and v2 api, here is a example https://gist.github.com/NHZlX/204da5e89a1263ab6953a669401e37d5 , and this need to be discussed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I temporarily remove this config file.

Copy link
Contributor

@NHZlX NHZlX left a comment

Choose a reason for hiding this comment

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

LGTM

@NHZlX NHZlX merged commit 0d6dfa3 into PaddlePaddle:master Oct 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants