Skip to content
New issue

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

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

Already on GitHub? # to your account

Add double sided field to material #130

Merged
merged 1 commit into from
Jan 30, 2021
Merged

Add double sided field to material #130

merged 1 commit into from
Jan 30, 2021

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Jan 30, 2021

Add double_sided field to material.proto

This field has already been added to the sdf spec in gazebosim/sdformat#410

related pull request in ign-gazebo that uses this field: gazebosim/gz-sim#599

Signed-off-by: Ian Chen ichen@osrfoundation.org

Signed-off-by: Ian Chen <ichen@osrfoundation.org>
@codecov
Copy link

codecov bot commented Jan 30, 2021

Codecov Report

Merging #130 (4a0e561) into main (249f1ec) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   84.78%   84.78%           
=======================================
  Files           7        7           
  Lines         828      828           
=======================================
  Hits          702      702           
  Misses        126      126           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 249f1ec...c6d3a88. Read the comment docs.

@iche033 iche033 changed the title add double sided field to material Add double sided field to material Jan 30, 2021
@chapulina chapulina merged commit 446fc89 into main Jan 30, 2021
@chapulina chapulina deleted the material_double_sided branch January 30, 2021 00:39
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🏢 edifice Ignition Edifice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants