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

Does AMCPruner only support 'mobilenet' and 'mobilenetv2' currently? #3632

Closed
ichejun opened this issue May 11, 2021 · 2 comments · Fixed by #3647
Closed

Does AMCPruner only support 'mobilenet' and 'mobilenetv2' currently? #3632

ichejun opened this issue May 11, 2021 · 2 comments · Fixed by #3647

Comments

@ichejun
Copy link
Contributor

ichejun commented May 11, 2021

Hi!
Does AMCPruner only support 'mobilenet' and 'mobilenetv2' currently? Some comments in the code are confusing:

model type to prune, currently 'mobilenet' and 'mobilenetv2' are supported.

#2876
and seems it already support resnet?

from torchvision.models import resnet

These makes me confused, does AMCPruner support network structures other than 'mobilenet' and 'mobilenetv2' currently?

If yes, can I use it in more other network structures, or is there any restrictions of net when using the amc pruner?
If not, is the share index for residual connection here limits the expansion of the model? or is there any other restrictions?

if self.args.model_type == 'mobilenetv2': # TODO: to be tested! Share index for residual connection

Thanks~

@J-shang
Copy link
Contributor

J-shang commented May 13, 2021

Yes, it supports resnet. You could pull requests to modify the comments if it is convenient. : )

Because AMCPruner develops based on mobilenet, mobilenetv2 open-source implementation, it is hard to expanse to other models.

@ichejun
Copy link
Contributor Author

ichejun commented May 15, 2021

@J-shang OK, thanks~ I pull request to modify the comments in #3647, and going to try other nets maybe :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants