Skip to content

Added light intensity#612

Merged
chapulina merged 5 commits intomainfrom
ahcorde/feature/intensity
Feb 12, 2021
Merged

Added light intensity#612
chapulina merged 5 commits intomainfrom
ahcorde/feature/intensity

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 ahcorde@gmail.com

Signed-off-by: ahcorde <ahcorde@gmail.com>
@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.

5 participants