search

wzhouxiff / RestoreFormerPlusPlus

[TPAMI2023] RestoreFormer++
Apache License 2.0
183 stars 8 forks source link

A small question... #10

Open 815056753qqcom opened 10 months ago

815056753qqcom commented 10 months ago

When the version of Basicsr is 1.4.2, "ModuleNotFoundError: No module named 'basicsr.losses.losses'"

When the version of Basicsr is 1.3.5, "RestoreFormer/modules/losses/vqperceptual.py", line 60. TypeError: '>' not supported between instances of 'ListConfig' and 'int'", I saw that in the RestoreFormerPlusPlus.yaml file, comp_weight is a list, It should be an int?

eblis88 commented 9 months ago

如果你是使用scripts/test.sh命令的话，应该要把basicsr版本切到1.3.4.4，他readme写错了 第二个错误是传参数的model_path写错了，需要手动修改以下。

如果是使用根目录下的inreference.py的话，要把basicsr版本切到1.4.2。而且那个inreference.py的第99行有个url = none要删掉，不然会报错

wzhouxiff commented 9 months ago

Thanks for your reminder. The latest version has revised the reported issues and aligned to Basicsr==1.4.2.