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 null checks when loading example render engine #156

Merged

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Oct 1, 2020

The null checks I added are in BaseRenderEngine.cc

iche033 and others added 3 commits September 30, 2020 21:52
Signed-off-by: Ian Chen <[email protected]>
Signed-off-by: John Shepherd <[email protected]>
Signed-off-by: John Shepherd <[email protected]>
@JShep1 JShep1 merged commit 12ec382 into jshep1/add_custom_plugin_example Oct 1, 2020
@JShep1 JShep1 deleted the ian/add_custom_plugin_example branch October 1, 2020 19:10
iche033 added a commit that referenced this pull request Oct 13, 2020
* Add unversioned symlinks for engine plugins

Signed-off-by: John Shepherd <[email protected]>

* Add init attempt at custom render engine

Signed-off-by: John Shepherd <[email protected]>

* Refactor

Signed-off-by: John Shepherd <[email protected]>

* Clean up

Signed-off-by: John Shepherd <[email protected]>

* Add init attempt

Signed-off-by: John Shepherd <[email protected]>

* Clean

Signed-off-by: John Shepherd <[email protected]>

* Add null checks when loading example render engine (#156)

Add null checks and docs

Signed-off-by: Ian Chen <[email protected]>

Co-authored-by: John Shepherd <[email protected]>

* Add windows build rules

Signed-off-by: John Shepherd <[email protected]>

* Take in paths from ign gazebo

Signed-off-by: John Shepherd <[email protected]>

* Update docs

Signed-off-by: John Shepherd <[email protected]>

* Add init attempt at rendering plugin tutorial

Signed-off-by: John Shepherd <[email protected]>

* Requested fixes

Signed-off-by: John Shepherd <[email protected]>

* minor tweak to heading

Signed-off-by: Ian Chen <[email protected]>

Co-authored-by: Ian Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants