search

PaddlePaddle / Mobile

Embedded and Mobile Deployment
Apache License 2.0
71 stars 29 forks source link

Refine `models` directory #55

Closed hedaoyuan closed 6 years ago

hedaoyuan commented 6 years ago

Fix #54

Xreki commented 6 years ago

I agree to reorganize the directory, but I am not sure about the sub-directories' name.

Xreki commented 6 years ago

In fact, I think the sub-directories' name under deployment is not intuitive. Since #54 is merged, we may keep it and change it in next PR.

About the sub-directory under models, how about change configuration to standard_network_configs?

hedaoyuan commented 6 years ago

About the sub-directory under models, how about change configuration to standard_network_configs?

OK. I will change the directory name to standard_network, and also we can add a README.md to explain the configuration is follow which article.

Xreki commented 6 years ago

we can add a README.md to explain the configuration is follow which article.

Yes, indeed we need to add this. Maybe @NHZlX add this in next PR?