Metalhead.jl icon indicating copy to clipboard operation
Metalhead.jl copied to clipboard

change pretrain to pretrained?

Open CarloLucibello opened this issue 2 years ago • 2 comments

Should we have

VGG(11, pretrained=true)

instead of

VGG(11, pretrain=true)

The latter seems a command...

CarloLucibello avatar Apr 23 '23 06:04 CarloLucibello

Even better, as in torchvision we could have a weights argument instead where we can pass more specific values, in case of multiple pretraining options:

VGG(11, weights="IMAGENET1K_V1")

CarloLucibello avatar Apr 23 '23 06:04 CarloLucibello

Yes the loading logic already supports that, and the plan was to move to that once we have multiple weight options.

darsnack avatar Apr 23 '23 09:04 darsnack