Closed xiankgx closed 2 years ago
Hi, thanks for printing this out. Yes, some of the parameters in the attention module is currently redundant, and you can safely remove them. This is the same for the second issue you pointed out.
Thank you author, I'm training a places2 model, hope it will work out well so I can wrap up my work.
link
Two attributes 'stride' and 'prop_kernels' doesn't seem to be used. Can I remove them?
link 2
Line 150 seems to be redundant with the existence of line 151, please correct me if I'm wrong.