big_vision icon indicating copy to clipboard operation
big_vision copied to clipboard

[BUG] in big_vision.models.proj.flexi.vit

Open KennyFu1997 opened this issue 2 years ago • 1 comments

Hello, big_vision team! Thanks for your work on the repository. I found two small typo in the flexivit code:

line 194 restored_params = utils.load_params(None, init_file) ==> restored_params = utils.load_params(init_file)

line 205 restored_params["embedding"]["kernel"] = resample_patchemb(old=old_patchemb, new_hw=model_cfg.patch_size) ==> restored_params["embedding"]["kernel"] = resample_patchemb(old=old_patchemb, new_hw=model_cfg.get("patch_size"))

KennyFu1997 avatar Nov 16 '23 22:11 KennyFu1997

Hi!

Thanks for flagging!

Flexivit project is currently not in sync with the head. Consider fixing bugs in your local fork for now and/or syncing back to the original commit that introduced Flexvit. I hope we find the capacity to update it in the near future, but can not promise.

akolesnikoff avatar Nov 22 '23 14:11 akolesnikoff