CRFP icon indicating copy to clipboard operation
CRFP copied to clipboard

the size is wrong

Open Winnie202 opened this issue 3 years ago • 3 comments

Thank you very much for your code contribution, after I run according to your code,like this: 2022-12-30 16-32-13屏幕截图

the test data shape is reds, lr image shape is 3232, hr image shape is 128128, but an error occurred,like this:

2022-12-30 16-36-13屏幕截图 I try to modify some parameters,in model/CRFP.py:

1

2022-12-30 16-42-56屏幕截图

3

but an error occurred,like this: 2022-12-30 16-46-44屏幕截图

Excuse me, where did I go wrong

Winnie202 avatar Dec 30 '22 08:12 Winnie202

Would it make sense to use the following argument?

--GT_size 128

in order not use the default value of 256:

https://github.com/eugenelet/CRFP/blob/c7d0b82735514ba182c14f188f29fdec390d6a6f/option.py#L88-L89

woctezuma avatar Jan 01 '23 12:01 woctezuma

Would it make sense to use the following argument?

--GT_size 128

in order not use the default value of 256:

https://github.com/eugenelet/CRFP/blob/c7d0b82735514ba182c14f188f29fdec390d6a6f/option.py#L88-L89

No, after the parameters are modified:--GT_size 128, an error is still reported

2023-01-02 12-24-11屏幕截图

Winnie202 avatar Jan 02 '23 04:01 Winnie202

Would it make sense to use the following argument?

--GT_size 128

in order not use the default value of 256: https://github.com/eugenelet/CRFP/blob/c7d0b82735514ba182c14f188f29fdec390d6a6f/option.py#L88-L89

No, after the parameters are modified:--GT_size 128, an error is still reported

2023-01-02 12-24-11屏幕截图

Hi, could you please check TestSet() class in dataset/reds.py? Since we source the different preprocessed LR dataset with different scaling factor. And in test mode we use full size of data and the parameters --GT_size and --LR_size will not be used. The size could be wrong at the begining when you load those data. image And I also wonder what are the shapes of these tensors in your case? Thank you.

lienfeng1011 avatar Jan 08 '23 08:01 lienfeng1011