icandle / CAMixerSR

CAMixerSR: Only Details Need More “Attention” (CVPR 2024)
https://arxiv.org/abs/2402.19289
Apache License 2.0
209 stars 11 forks source link

basicsr文件中缺少各种py文件 #28

Open banjamn opened 2 weeks ago

banjamn commented 2 weeks ago

作者你好,非常感谢你能公开源码。我在测试你的代码的时候发现basicsr文件夹中很多文件没有包含,当我一一复制进去的时候又报错了。例如当我复制进去vgg_arch.py进去之后,再次运行时报错, image 。所以希望作者能够提供更加完整的代码。谢谢。

icandle commented 2 weeks ago

我看了一下这应该是因为basicsr这个文件夹的名称和basicsr库冲突没有调用basicsr库中的文件,一个简单的办法是吧codes中basicsr的文件名改为camixer,我们将在后续的更新中修正这一点。

Kohaneee commented 2 weeks ago

我看了一下这应该是因为basicsr这个文件夹的名称和basicsr库冲突没有调用basicsr库中的文件,一个简单的办法是吧codes中basicsr的文件名改为camixer,我们将在后续的更新中修正这一点。

image image

我把文件名字改成您给的,还是出现了这样的错误,我感觉是您这边的写的models/init.py和archs/init.py没有用上导致的注册问题,后续重新整理一下类的继承大概能解决

icandle commented 2 weeks ago

我看了一下这应该是因为basicsr这个文件夹的名称和basicsr库冲突没有调用basicsr库中的文件,一个简单的办法是吧codes中basicsr的文件名改为camixer,我们将在后续的更新中修正这一点。

image image

我把文件名字改成您给的,还是出现了这样的错误,我感觉是您这边的写的models/init.py和archs/init.py没有用上导致的注册问题,后续重新整理一下类的继承大概能解决

这个问题可能还是跟具体的环境有关,这两个error我都能复现,对于第二个问题可以参考hat issue 28,最稳的方法应该就是将models和archs的文件放在本地的basicsr里面。