Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible bug in render_pipeline/render_model_views.py #5

Open
cache-tlb opened this issue Aug 30, 2016 · 0 comments
Open

Possible bug in render_pipeline/render_model_views.py #5

cache-tlb opened this issue Aug 30, 2016 · 0 comments

Comments

@cache-tlb
Copy link

cache-tlb commented Aug 30, 2016

I'm trying to following the instructions in README.md and in section Off-the-shelf Viewpoint Estimator I can get the correct output as in ref_output/est-view.txt shows when I run run_demo.py, which is 228 11 8.

However, when I run run_visualize_3dview.py, the generated image aeroplane_in_estimated_view.png is different with the reference image, as shown below. The pitch angle differs a lot.
aeroplane_in_estimated_view

I'm using 64bit Linux and blender 2.71 as the instruction says. I execute the output command that calls blender, which looks like
/usr/local/blender/blender /home/ZZ/RenderForCNN-master/render_pipeline/blank.blend --background --python /home/ZZ/RenderForCNN-master/render_pipeline/render_model_views.py -- /home/ZZ/RenderForCNN-master/demo_render/sample_model2/model.obj xxx xxx ./view.txt .. This produces the same result. (Of course I make a fake view.txt in that folder whose content is 228.0 11.0 8.0 1.0.)
I think there may be something wrong with the blender file render_pipeline/render_model_views.py.
I notice in this file, line 209 writes
theta_deg = (-1*theta_deg)%360
and line 198 writes
q2 = camRotQuaternion(cx, cy, cz, theta_deg).
I change the line 198 to
q2 = camRotQuaternion(cx, cy, cz, -1*theta_deg)
and get the correct result. Could you please check it?

@cache-tlb cache-tlb changed the title Rendered image is much different from the reference output in demo_view Probable mistake in render_pipeline/render_model_views.py Aug 31, 2016
@cache-tlb cache-tlb changed the title Probable mistake in render_pipeline/render_model_views.py Possible bug in render_pipeline/render_model_views.py Aug 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant