Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pretrained Encoder link #2

Closed
BrianPugh opened this issue Mar 24, 2021 · 8 comments
Closed

Pretrained Encoder link #2

BrianPugh opened this issue Mar 24, 2021 · 8 comments

Comments

@BrianPugh
Copy link

BrianPugh commented Mar 24, 2021

In the README, the pretrained encoder cmd should be:

wget https://storage.googleapis.com/vit_models/imagenet21k/R50%2BViT-B_16.npz

not the current:

wget https://storage.googleapis.com/vit_models/imagenet21k/R50-ViT-B_16.npz
@BrianPugh
Copy link
Author

while we're at it, the mkdir command should be:

mkdir -p ./model/vit_checkpoint/imagenet21k 

@BrianPugh
Copy link
Author

finally, the last command should be:

mv R50+ViT-B_16.npz model/vit_checkpoint/imagenet21k/

confirmed by the config you are using here

@BrianPugh
Copy link
Author

and the prepare nyu commands can be shorted to:

mkdir -p pytorch/dataset/nyu_depth_v2
python utils/download_from_gdrive.py 1AysroWpfISmm-yRFGBgFTrLy6FjQwvwP pytorch/dataset/nyu_depth_v2/sync.zip
cd pytorch/dataset/nyu_depth_v2
unzip sync.zip

@ygjwd12345
Copy link
Owner

In the README, the pretrained encoder cmd should be:

wget https://storage.googleapis.com/vit_models/imagenet21k/R50%2BViT-B_16.npz

not the current:

wget https://storage.googleapis.com/vit_models/imagenet21k/R50-ViT-B_16.npz

it is according to ViT

imagenet21k pre-train

wget https://storage.googleapis.com/vit_models/imagenet21k/{MODEL_NAME}.npz

@ygjwd12345
Copy link
Owner

and the prepare nyu commands can be shorted to:

mkdir -p pytorch/dataset/nyu_depth_v2
python utils/download_from_gdrive.py 1AysroWpfISmm-yRFGBgFTrLy6FjQwvwP pytorch/dataset/nyu_depth_v2/sync.zip
cd pytorch/dataset/nyu_depth_v2
unzip sync.zip

Thanks

@BrianPugh
Copy link
Author

the MODEL_NAME has a +, not a - after R50

@ygjwd12345
Copy link
Owner

Sorry it's my mistake

@ygjwd12345
Copy link
Owner

the MODEL_NAME has a +, not a - after R50

It has be fixed.

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

No branches or pull requests

2 participants