Light Commands via topic

[WilliamLewww]

🎉 Light Commands

Summary

Added a user command to control light properties:

ign topic -t /world/default/light_config -m Light -p "name: 'pointlight', diffuse: {r: 1.0, g: 0.0, b: 0.0, a: 1.0}, specular: {r: 1.0, g: 0.0, b: 0.0, a: 1.0}"

Added a light controller to control a specific light's properties:

ign topic -t /light/left_light_source/light_config -m Light -p "diffuse: {r: 1.0, g: 0.0, b: 0.0. a: 0.0}"

Checklist

• Signed all commits for DCO
• Added tests
• Added example and/or tutorial
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

[codecov]

Codecov Report

Merging #1222 (691384c) into ign-gazebo6 (0810a6b) will increase coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 691384c differs from pull request most recent head 8b224bc. Consider uploading reports for the commit 8b224bc to get more accurate results

@@               Coverage Diff               @@
##           ign-gazebo6    #1222      +/-   ##
===============================================
+ Coverage        61.95%   61.97%   +0.01%     
===============================================
  Files              276      276              
  Lines            22991    23000       +9     
===============================================
+ Hits             14245    14254       +9     
  Misses            8746     8746              

Impacted Files	Coverage Δ
src/systems/user_commands/UserCommands.cc	54.02% <100.00%> (+0.63%)	⬆️

---

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 0810a6b...8b224bc. Read the comment docs.

[WilliamLewww]

Ignore the review request,

Trying to fix the DCO caused a bunch of new error. I'm currently trying to revert the changes.

[chapulina]

Looking good. We should add a test using the topic here:

https://github.com/ignitionrobotics/ign-gazebo/blob/17f63c497e7fb81dca5f87708ba8b2128db67b31/test/integration/user_commands.cc#L692

[chapulina]

LGTM, just some minor comments before merging

Addressed

[osrf-triage]

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-ignition-releases-2022-01-24-citadel-edifice-fortress/1241/1