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

Add light intensity field to ComponentInspector #670

Merged
merged 2 commits into from
Mar 10, 2021
Merged

Add light intensity field to ComponentInspector #670

merged 2 commits into from
Mar 10, 2021

Conversation

atharva-18
Copy link
Contributor

@atharva-18 atharva-18 commented Mar 6, 2021

Signed-off-by: Atharva Pusalkar [email protected]

🎉 New feature

Closes #637

Summary

I have added light intensity field to the ComponentInspector. Changes were also made to RenderUtil.cc and UserCommands.cc to set the light intensity value.
Here's a demo:
light_intensity

Test it

You can test the feature by running ign gazebo examples/worlds/lights.sdf

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

@github-actions github-actions bot added the 🏢 edifice Ignition Edifice label Mar 6, 2021
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for the contribution!

Do you mind to add the intensity to the Plotting plugin ?

Signed-off-by: Atharva Pusalkar <[email protected]>
@atharva-18
Copy link
Contributor Author

@ahcorde Added in b145cb4!

@chapulina chapulina added the rendering Involves Ignition Rendering label Mar 8, 2021
@ahcorde
Copy link
Contributor

ahcorde commented Mar 9, 2021

@osrf-jenkins retest this please

Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting CI

Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@ahcorde
Copy link
Contributor

ahcorde commented Mar 10, 2021

@osrf-jenkins retest this please

@atharva-18
Copy link
Contributor Author

These tests are failing on macOS catalina:

The following tests FAILED:
	 83 - INTEGRATION_imu_system (SEGFAULT)
	113 - INTEGRATION_multiple_servers (SEGFAULT)
	139 - INTEGRATION_user_commands (SEGFAULT)

@ahcorde ahcorde merged commit f1bc04b into gazebosim:main Mar 10, 2021
@ahcorde
Copy link
Contributor

ahcorde commented Mar 10, 2021

Thank you for the contribution @atharva-18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏢 edifice Ignition Edifice rendering Involves Ignition Rendering
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow to modify light intensity with the ComponentInspector - ign-gazebo5
3 participants