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

Exit early if deployments are not allowed #88

Merged
merged 16 commits into from
May 7, 2020

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Apr 23, 2020

This prevents a topic from creation, and also limits some computation.

src/systems/breadcrumbs/Breadcrumbs.cc Outdated Show resolved Hide resolved
@chapulina
Copy link
Contributor

@nkoenig nkoenig force-pushed the breadcrumbs_exit_early branch from 3fb0ace to ea03e8d Compare May 6, 2020 22:09
@nkoenig
Copy link
Contributor Author

nkoenig commented May 6, 2020

Documentation has been updated.

claireyywang and others added 13 commits May 6, 2020 15:19
Signed-off-by: claireyywang <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: claireyywang <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
* Add and use entityCast helper function

* return early for missing features

* move minimum map out of entityCast

Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
* [ign-gazebo2] Update BitBucket links

Signed-off-by: Louise Poubel <[email protected]>

* [ign-gazebo2] Update BitBucket links

Signed-off-by: Louise Poubel <[email protected]>

* Apply suggestions from code review

Co-authored-by: Marya Belanger <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
…, such as, ign-launch

Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
The crash was caused by an access to an invalid address within Qt when
trying to access the command line arguments to the application. These
are passed as argc and argv to QApplication, but for argc, we were
passing a reference to a local.

Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
@nkoenig nkoenig force-pushed the breadcrumbs_exit_early branch from ded47ae to b8ac88b Compare May 6, 2020 22:21
Nate Koenig added 2 commits May 6, 2020 15:22
@nkoenig nkoenig merged commit 9a3b42b into ign-gazebo2 May 7, 2020
@nkoenig nkoenig deleted the breadcrumbs_exit_early branch March 19, 2021 22:40
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.

4 participants