wyf0912 / LLFlow

[AAAI 2022] The code release of paper "AAAI Low-Light Image Enhancement with Normalizing Flow"
Other
289 stars 36 forks source link

Questions about the paper #21

Closed ph0316 closed 2 years ago

ph0316 commented 2 years ago

Hello, I'd like to ask you two questions. The first: why should the input low light image be enhanced by histogram first? Can the low light image be directly sent to the network? Second: you said that you can learn one-to-many using normalizing flow. Where is it?

wyf0912 commented 2 years ago
  1. Our model can directly use raw low-light images as input. Proper hand-crafted features as input will further benefit the robustness.

  2. You can manually modify the input $z$ in the code to get different $x_h$

ph0316 commented 2 years ago

image I can't train when I'm using ’NoEncoder‘?Why is that?

wyf0912 commented 2 years ago

That encoder is just for ablation and is out of maintenance.

Maybe you could try to modify the following code from

elif opt['cond_encoder'] ==  'NoEncoder':
    self.RRDB = None # NoEncoder(in_nc, out_nc, nf, nb, gc, scale, opt)

to

elif opt['cond_encoder'] ==  'NoEncoder':
    self.RRDB = NoEncoder(in_nc, out_nc, nf, nb, gc, scale, opt)
wyf0912 commented 2 years ago

I am not sure whether this works. If you want to try "NoEncoder", the easiest way is to make the current encoder just output 0.

That encoder is just for ablation and is out of maintenance.

Maybe you could try to modify the following code from

elif opt['cond_encoder'] ==  'NoEncoder':
    self.RRDB = None # NoEncoder(in_nc, out_nc, nf, nb, gc, scale, opt)

to

elif opt['cond_encoder'] ==  'NoEncoder':
    self.RRDB = NoEncoder(in_nc, out_nc, nf, nb, gc, scale, opt)
ph0316 commented 2 years ago

I have tried to do this, but the picture is black.

wyf0912 commented 2 years ago

You could refer to this issue #18 . It's very difficult to only train the invertible network without conditional features.

ph0316 commented 2 years ago

Thank you very much!