-
Notifications
You must be signed in to change notification settings - Fork 467
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
Animating camera leads to unexpected results #444
Comments
@lijenicol I think the culprit is the 'Scale -1 1 1' statement in your scene.. try to remove and render again |
@pbrt4bounty Yes that |
Basically, I removed this in my Pbrt-v4 for Blender integration. 'Scale -1 1 1' is not anymore on the scenes created from this plugin. |
It's in the original scenes because there was a bug in some of the camera code in pbrt-v1 (and maybe pbrt-v2?) that, when corrected, flipped images horizontally compared to before. So that got added to all of the scene files so renderings wouldn't change. I agree that the behavior @lijenicol is seeing is a bug; there's no reason that shouldn't work. I haven't had a chance to dig into the code yet but hope do to so later in December. |
Thanks @mmp, let us know what you find! |
To be honest.. going through my code, I have found a single point where I am forced to use Scale -1 1 1: Spherical Camera
Now I don't remember how much time I spent on this part of the code, but maybe there is another solution that doesn't need Scale -1 1 1 |
Consider the following pbrt scene (modified off the example here), where the transformations for the camera at the start and end times are the same, and the x-axis is flipped using
Scale
:That produces this image:
Now, modify the start or end time transform slightly, and all of a sudden the camera flips unexpectedly. In this case I changed the end time from above by adding 0.1 to the x-axis:
With that adjustment, I get:
This oddity seems to only happen when any of the
Scale
axes are negative. If all the scale axes are positive, then the animated camera doesn't do the strange flip.(My pbrt version is based off the most recent commit on master:
39e01e6
)The text was updated successfully, but these errors were encountered: