hustvl / ViTMatte

[Information Fusion (Vol.103, Mar. '24)] Boosting Image Matting with Pretrained Plain Vision Transformers
MIT License
339 stars 33 forks source link

about detectron2 version error ? I use source code to install detectron2-0.6(https://github.com/facebookresearch/detectron2/releases/tag/v0.6) #6

Closed zhanghongyong123456 closed 1 year ago

zhanghongyong123456 commented 1 year ago

v0.6 version batch_norm doesn't have the 'LN' method?

image

I compared the latest version of the code with version 0.6 and found that this method was indeed missing

image image

JingfengYao commented 1 year ago

Thanks for your observation. I've checked the d2 version in my environment and it is shown as follows: image But I just built d2 from the latest source code instead of the released pre-built v0.6 one. So I checked the source code in d2 and found that its version is fixed at 0.6 since two years ago, which caused the misunderstanding.

image

For ViTMatte, you could build d2 from the latest source codes. If it still does not work, please let me know and I'll try to help.

zhanghongyong123456 commented 1 year ago

Thanks for your observation. I've checked the d2 version in my environment and it is shown as follows: image But I just built d2 from the latest source code instead of the released pre-built v0.6 one. So I checked the source code in d2 and found that its version is fixed at 0.6 since two years ago, which caused the misunderstanding. image For ViTMatte, you could build d2 from the latest source codes. If it still does not work, please let me know and I'll try to help.

it is OK , use latest(main) code, I recommend stating on the project home page that using the latest code (especially for windows users, source code compilation is required)

JingfengYao commented 1 year ago

@zhanghongyong123456 We have updated it. Thanks for your suggestion. 😄

zhanghongyong123456 commented 1 year ago

@zhanghongyong123456 We have updated it. Thanks for your suggestion. 😄

Thanks again for a great project