-
Notifications
You must be signed in to change notification settings - Fork 51
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
[Ogre2] Fix invalid anti-aliasing level warning #470
[Ogre2] Fix invalid anti-aliasing level warning #470
Conversation
- Fix a bug where the warning of an invalid anti-aliasing level in Ogre2RenderTarget was not being output - Set the FSAA level to 0 if the target level is invalid - Provide a list of valid options in the warning Signed-off-by: Rhys Mainwaring <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-rendering6 #470 +/- ##
==================================================
+ Coverage 53.50% 53.51% +0.01%
==================================================
Files 192 192
Lines 19562 19569 +7
==================================================
+ Hits 10466 10473 +7
Misses 9096 9096
Continue to review full report at Codecov.
|
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
retriggered a couple of CI builds that failed for unrelated reason. Merging this once CI is green |
- Fix a bug where the warning of an invalid anti-aliasing level in Ogre2RenderTarget was not being output - Set the FSAA level to 0 if the target level is invalid - Provide a list of valid options in the warning Signed-off-by: Rhys Mainwaring <[email protected]>
- Fix a bug where the warning of an invalid anti-aliasing level in Ogre2RenderTarget was not being output - Set the FSAA level to 0 if the target level is invalid - Provide a list of valid options in the warning Signed-off-by: Rhys Mainwaring <[email protected]> Signed-off-by: William Lew <[email protected]>
🦟 Bug fix
Fixes issue discussed in #463 (comment)
Summary
This PR fixes a bug where the invalid anti-aliasing level warning in
Ogre2RenderTarget::TargetFSAA()
was not being output and the FSAA level was not being set to a default value of 0. This can cause the scene to fail to render.The code will now output a warning, provide a list of valid options and set the anti-aliasing level to 0.
The code can be tested by changing the camera anti-aliasing level in the
simple_demo
example:Modify:
https://github.com/ignitionrobotics/ign-rendering/blob/1c77a8ac05cce3ed5a5b99e56030db833ad67b0a/examples/simple_demo/Main.cc#L179
to
The console output is:
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge