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 light intensity #612

Merged
merged 5 commits into from
Feb 12, 2021
Merged

Added light intensity #612

merged 5 commits into from
Feb 12, 2021

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Feb 4, 2021

Added light intensity. Related with this issue gazebosim/gz-rendering#232

Need some releases:

Signed-off-by: ahcorde [email protected]

Signed-off-by: ahcorde <[email protected]>
@ahcorde ahcorde added needs upstream release Blocked by a release of an upstream library 🏢 edifice Ignition Edifice labels Feb 4, 2021
@ahcorde ahcorde requested a review from iche033 February 4, 2021 18:01
@ahcorde ahcorde requested a review from chapulina as a code owner February 4, 2021 18:01
@ahcorde ahcorde self-assigned this Feb 4, 2021
@ahcorde
Copy link
Contributor Author

ahcorde commented Feb 8, 2021

@osrf-jenkins retest this please

@iche033
Copy link
Contributor

iche033 commented Feb 8, 2021

@codecov
Copy link

codecov bot commented Feb 9, 2021

Codecov Report

Merging #612 (ffe3f13) into main (8f1f44f) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #612      +/-   ##
==========================================
- Coverage   77.43%   77.33%   -0.10%     
==========================================
  Files         212      212              
  Lines       12016    12018       +2     
==========================================
- Hits         9304     9294      -10     
- Misses       2712     2724      +12     
Impacted Files Coverage Δ
src/Conversions.cc 83.18% <100.00%> (+0.04%) ⬆️
src/SimulationRunner.cc 91.68% <0.00%> (-2.17%) ⬇️

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 8f1f44f...ffe3f13. Read the comment docs.

@ahcorde
Copy link
Contributor Author

ahcorde commented Feb 11, 2021

@osrf-jenkins retest this please

@ahcorde
Copy link
Contributor Author

ahcorde commented Feb 11, 2021

@osrf-jenkins retest this please

1 similar comment
@ahcorde
Copy link
Contributor Author

ahcorde commented Feb 11, 2021

@osrf-jenkins retest this please

@chapulina chapulina removed the needs upstream release Blocked by a release of an upstream library label Feb 11, 2021
@ahcorde
Copy link
Contributor Author

ahcorde commented Feb 11, 2021

@iche033 CI is green!

@ahcorde ahcorde requested a review from iche033 February 11, 2021 19:11
@chapulina chapulina merged commit a5e340f into main Feb 12, 2021
@chapulina chapulina deleted the ahcorde/feature/intensity branch February 12, 2021 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏢 edifice Ignition Edifice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants