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

Added shininess to models #3213

Merged
merged 7 commits into from
May 6, 2022

Conversation

WilliamLewww
Copy link
Contributor

Depends on:

When using default shaders in gazebo, the ogre's default shininess is 0 causing the specular components to have no effect. Without having a shininess element, we are unable to utilize specular highlights with default shaders.

Test it:

gazebo worlds/shapes_shininess.world

Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
gazebo/physics/World.cc Outdated Show resolved Hide resolved
gazebo/physics/World.hh Outdated Show resolved Hide resolved
gazebo/rendering/Visual.cc Outdated Show resolved Hide resolved
gazebo/rendering/Visual.cc Outdated Show resolved Hide resolved
gazebo/rendering/Visual.cc Outdated Show resolved Hide resolved
gazebo/rendering/Visual.hh Outdated Show resolved Hide resolved
Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
gazebo/rendering/Visual.cc Show resolved Hide resolved
gazebo/physics/World.cc Outdated Show resolved Hide resolved
gazebo/rendering/Visual.cc Outdated Show resolved Hide resolved
gazebo/rendering/Visual.cc Outdated Show resolved Hide resolved
Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
@WilliamLewww WilliamLewww merged commit f6d2b94 into gazebosim:gazebo11 May 6, 2022
scpeters added a commit to gazebo-release/gazebo11-release that referenced this pull request May 27, 2022
Follow-up to gazebosim/gazebo-classic#3213 and gazebosim/gazebo-classic#3223

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
scpeters added a commit to gazebo-release/gazebo11-release that referenced this pull request May 27, 2022
Follow-up to gazebosim/gazebo-classic#3213 and gazebosim/gazebo-classic#3223

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
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

Successfully merging this pull request may close these issues.

2 participants