cypw / PyTorch-MFNet

MIT License
253 stars 56 forks source link

Some questions and a suggestion #1

Closed vinsis closed 6 years ago

vinsis commented 6 years ago

Hi Yunpeng, just read your paper and have a couple of quick questions:

# current version: num_ix = int(num_mid/4)
num_ix = int(num_in/4)

I believe self.conv_i1 and self.conv_i2 are the layers for the multiplexer. Or am I getting it wrong?

Lastly, a small suggestion: since you are using PyTorch v0.4, you need not use Variable anymore. Hence you can write this line as:

data = torch.randn(1,3,16,224,224)

Thank you.

cypw commented 6 years ago

Hi @vinsis,

Thank you for the questions.

Thank you.

vinsis commented 6 years ago

Thanks @cypw for replying. Closing the issue now.