Skip to content

Merge with Upstream / Style Fixes / Modularize Method #1

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.

Sign up for GitHub

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

Jump to bottom
Closed
DLu wants to merge 32 commits into from
Closed

Merge with Upstream / Style Fixes / Modularize Method #1

DLu wants to merge 32 commits into from

Conversation

@DLu
Copy link

@DLu DLu commented Mar 7, 2019

First, my apologies for sitting on locusrobotics#9 for so long.

To get things moving, I've done three things.

  1. I merged your branch with upstream
  2. I changed the code style to be consistent with the standard roslint style
  3. I modularized out the closest point operation (based on code that I've written in the intervening time since you made the PR)

Once we get this merged, we can talk more substantially about the PR.

DLu and others added 30 commits October 23, 2018 11:52
@DLu
Nav Grid Package (#12)
71dc18f
@DLu
Change nav_core2 Interfaces (#13)
a671c6d 
* Change nav_core2 interfaces

* Update nav_core_adapter (with new CostmapAdapter)

* Update CostmapQueue

* Add Polygon/Footprint Support

* Update DWB to New Interface
@DLu
Nav Grid Iterators (#14)
c4c5cd1 
* Nav Grid Iterators package

* Use nav_grid_iterator in dwb_critics
@DLu
DWB Cleanup and Bug Fixes (#15)
d1d689a 
* New NavGridInfo msg and conversions

* General DWB Cleanup and bug fixes
@DLu
Various Nav Utils (#16)
fb4c4b2 
* New message for UIntBounds

* nav_2d_utils cleanup

* Nav Grid Pub Sub
@DLu
Global Planner (#17)
87ddd1c
@DLu
Metapackage and additional documentation (#18)
c8e2ee3
@DLu
0.2.1
0172e3a
@DLu
Fix Tests (#19)
4b8c6fe
@DLu
Fix node name resolving
b26c5f0
@DLu
Locomotor and related changes (#20)
8b44155
@DLu
Misc fixes: Fix unsigned int logic, change default global planner, ad…
34bf792 
…d some explicit header dependencies
@DLu
Use new service instead of imaginary std_srvs::SetString
428868b
@DLu
0.2.0
c571761
@DLu
Updated test for earlier versions of ROS
2d816ad
@DLu
Change default value in NavGrid to template-default
5611184
@DLu
Fix Buildfarm Tests (#23)
cbbfef5
@DLu
0.2.2
5564bf4
@DLu
Fix a bug with costmap_queue getNextCell (#25)
c24cfa9
@mintar @DLu
Fix buildfarm tests again (increase test timeout) (#30)
fbe15c3
@DLu
Update reset documentation (#31)
8248d6c
@DLu
Stricter tests and missing headers for bionic builds (#32)
7ca6831
@DLu
Misc global planning fixes (#33)
3f716c2
@DLu
Adjust plan resolution fix (#34)
9133e65 
* Add pose function

* Adjust plan resolution fix
@mintar @DLu
Fix build with catkin_make (#29)
9ab022f
@nlimpert @DLu
Catch exceptions by reference (#24)
394f306 
* global_planner: catch exceptions by reference

GCC 8 raises errors on exceptions caught by value.

* locomotor: catch by reference in ErrorHandling doc
@DLu
Fake version bump
41a95ee
@DLu
0.2.5
0a4e7ed
@hvpandya @DLu
Remove additional test directory (#36)
f9cab9c 
Additional 'test' directory gets created under 'test/' at install location, this breaks certain test launches
Merge remote-tracking branch 'origin/master' into nirzvi1_master
3772314
@DLu DLu closed this Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@DLu @mbuijs @mintar @nlimpert @hvpandya