DASR icon indicating copy to clipboard operation
DASR copied to clipboard

degradation params

Open jiamingNo1 opened this issue 2 years ago • 2 comments

I have two questions.

  1. the sinc kernel_size may be negative when the prob larger than final_sinc_prob? https://github.com/csjliang/DASR/blob/ff2e1ec02c767b75d09b5d60f85c5cbd4115d058/dasr/models/DASR_model.py#L106
  2. why are the previous degradation params overwritten , that is to say, the sinc degradation_params[:, 9:10] is overwritten by the second blur prob? https://github.com/csjliang/DASR/blob/ff2e1ec02c767b75d09b5d60f85c5cbd4115d058/dasr/models/DASR_model.py#L161

jiamingNo1 avatar Jun 06 '22 08:06 jiamingNo1

hello, I am also confused with the 'degradation_params', it seems that the images blurred by different sigma are get with the same 'degradation_params':

DASR_model.py#L100 self.degradation_params[:, self.road_map[0]+1:self.road_map[0]+2] = (data['kernel1']['sigma_x'].unsqueeze(1) - self.opt_train['blur_sigma'][0]) / (self.opt_train['blur_sigma'][1] - self.opt_train['blur_sigma'][0])

DASR_model.py#L257 self.degradation_params[:, self.road_map[0]+1:self.road_map[0]+2] = (data['kernel1']['sigma_x'].unsqueeze(1) - self.opt_train['blur_sigma_standard1'][0]) / (self.opt_train['blur_sigma_standard1'][1] - self.opt_train['blur_sigma_standard1'][0])

train_DASR.yml#L88 blur_sigma: [0.2, 3]

train_DASR.yml#L79 blur_sigma_standard1: [ 0.2, 1.5 ]

thus in DASR_model.py#L100: if sigma = 1, degradation_params[:, self.road_map[0]+1:self.road_map[0]+2] = (1 - 0.2)/(3 - 0.2) = 0.286

in DASR_model.py#L257 if sigma = 0.57, degradation_params[:, self.road_map[0]+1:self.road_map[0]+2] = (0.57 - 0.2)/(1.5 - 0.2) = 0.285

the sigma is quite different while the 'degradation_params' is quite similar, is this right?

yiyunchen avatar Jun 22 '22 02:06 yiyunchen