Replies: 2 comments
-
@zmurez I think changes like this could be better done with a helper fn that walks the model modules, kind of related to #1529 ... I'd rather explore feasbility of adding model manipulation helpers. Adding more kwargs for rarely used functionality increases maintenance burden and modelling code complexity (EDIT: not to mention, needs to add for every model that you want to support) For this I think it'd be pretty easy to iterate over all modules and change the padding mode for all conv > 1x1. Quick draft:
|
Beta Was this translation helpful? Give feedback.
0 replies
-
I like that solution. Thanks! |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I would like to create a resnet model with
padding_mode=='reflect'
for all conv layers. This would be possible if we added apadding_mode
kwarg throughout. I can submit a PR if this sounds reasonable?Beta Was this translation helpful? Give feedback.
All reactions