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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement consistent Auto Aspect Ratio #764
Implement consistent Auto Aspect Ratio #764
Conversation
The old default Aspect Ratio would be 1.3333 Now it is 0.0 which indicates Gazebo should autocalculate the Aspect Ratio. This results in a nice backwards-compatible behavior while fixing various bugs mentioned in gazebosim#763 Fixes gazebosim#763 Signed-off-by: Matias N. Goldberg <dark_sylinc@yahoo.com.ar>
Add AspectRatio unit test Highly reduce the error tolerance on Camera UNIT test Signed-off-by: Matias N. Goldberg <dark_sylinc@yahoo.com.ar>
Signed-off-by: Matias N. Goldberg <dark_sylinc@yahoo.com.ar>
Signed-off-by: Matias N. Goldberg <dark_sylinc@yahoo.com.ar>
42af876
to
7bdec8b
Compare
Codecov Report
@@ Coverage Diff @@
## gz-rendering7 #764 +/- ##
=================================================
+ Coverage 75.48% 75.49% +0.01%
=================================================
Files 164 164
Lines 14364 14374 +10
=================================================
+ Hits 10843 10852 +9
- Misses 3521 3522 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me, just one question about the debugger capture call
Signed-off-by: Matias N. Goldberg <dark_sylinc@yahoo.com.ar>
馃 Bug fix
Fixes #763
Summary
The old default Aspect Ratio would be 1.3333
Now it is 0.0 which indicates Gazebo should autocalculate the Aspect Ratio.
This results in a nice backwards-compatible behavior while fixing various bugs mentioned in #763
Fixes #763
Signed-off-by: Matias N. Goldberg dark_sylinc@yahoo.com.ar
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.