Giter Club home page Giter Club logo

Comments (21)

dmarnerides avatar dmarnerides commented on July 19, 2024

Hey, what do you mean by shadows? Is it the band on the right and bottom? Also, can you include your input image?

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

Hi, thank you for your reply.
The shadow I said means that the square inside looks darker.
input 00999999-4-inputs 1

output test_indoor

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

Hm.. I just ran it and the .hdr output looks fine. Did you use any other arguments when you ran it?

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

This is the (reinhard) tone mapped prediction I get.

prediction_reinhard

Which version of PyTorch are you using?

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

I did not use other arguments. I run this code according to the Readme, and the instruction is python expand.py ldr_input.jpg
And I am using the version of PyTorch is 0.3.1

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

This is weird. I am using version '1.0.0a0+bcb62cb' of PyTorch but things shouldn't be any different for torch > 0.3.

How are you creating the tone mapped version of the prediction that you attach here?

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

I think this problem has nothing to do with the tone mapping operator.
I tried the function tonemap in MATLAB, and there will still be obvious boundaries(as shown below).
tone1

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

I just installed a new environment with PyTorch 0.3.1 and I couldn't reproduce the problem. Have you tried rerunning it? It kinda looks like the operation was stopped midway.

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

Thank you very much, you are an enthusiastic person!!
I used pip install OpenCV not conda install opencv -c menpo. Does this have an impact on results?

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

No worries. Yeah that could cause some problems but I don't think that that's the problem in this case. Can you reproduce this effect if you run it for a second time?

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

I ran it for a second time and tested other inputs. It still has problem with boundaries.
input
00999999-5-inputs
output
indoor2

input
00999999-6-inputs

output
indoor3

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

does the problem remain if you use a different patch size?
e.g.
python expand.py img.jpg --patch_size 128

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

It seems smaller
55

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

It looks like there might be a bug with the patch indexing, I'll try and reproduce it/figure it out some time later today.

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

In the meantime, can you email me the actual images you are using as inputs? There might be some downsampling going on in the ones attached here. Or jist let me know the original dimensions

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

Ok. Thanks again.

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

Hey, I managed to reproduce the effect and i think i fixed it. Can you please try again and let me know?

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

The boundary still exists.
And I use python2.7 which not support print(,end=''), but it will not affect the results.

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

Hey, you should have opened with that :). Python 2.7 was not supported, but you are right, it wasn't mentioned anywhere.

I managed to reproduce exactly what you had in a Python 2.7 environment. The problem was division (and printing). I added the imports and it should work now. Please let me know.

from hdr-expandnet.

Jee-King avatar Jee-King commented on July 19, 2024

Hi, It works now! Thank you very much!
So the problem was division because of python vision.
Thanks again!
(reinhard)
indoor1_prediction_reinhard

from hdr-expandnet.

dmarnerides avatar dmarnerides commented on July 19, 2024

No worries, glad to help!

from hdr-expandnet.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.