Skip to content

merge transient_local_publish into iron #74

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
Merged
merged 33 commits into from
Aug 15, 2024
Merged

merge transient_local_publish into iron #74

merged 33 commits into from
Aug 15, 2024

Conversation

@papertowel123
Copy link

@papertowel123 papertowel123 commented Jul 24, 2024

Basic Info

Info Please fill out this column
Ticket(s) this addresses (add tickets here #1)
Primary OS tested on (Ubuntu, MacOS, Windows)
Robotic platform tested on (Steve's Robot, gazebo simulation of Tally, hardware turtlebot)

Description of contribution in a few bullet points

Description of documentation updates required from your changes

Future work that may be required in bullet points

For Maintainers:

  • Check that any new parameters added are updated in navigation.ros.org
  • Check that any significant change is added to the migration guide
  • Check that any new features OR changes to existing behaviors are reflected in the tuning guide
  • Check that any new functions have Doxygen added
  • Check that any new features have test coverage
  • Check that any new plugins is added to the plugins page
  • If BT Node, Additionally: add to BT's XML index of nodes for groot, BT package's readme table, and BT library lists

jplapp and others added 25 commits January 22, 2024 12:04
@jplapp
add second costmap
5528c59
@jplapp
Merge branch 'iron' into drive_with_obstacles
c19ac0f 
# Conflicts:
#	nav2_controller/src/controller_server.cpp
@jplapp
don't stop if TEB sends zero speed, fix build
e41d060
@jplapp
.
6c9bc4b
@jplapp
.
269395d
@jplapp
fix build, add narrow costmap
3abda97
@jplapp
Merge remote-tracking branch 'origin/iron' into drive_with_obstacles
db5a694
@jplapp
don't pass sensor costmap ros to controllers
4e353fb
@jplapp
revert progress checker changes
6e2ab02
@jplapp
.
cd0c681
@jplapp
add function updateWithMin
10e011c
@jplapp
Merge pull request #69 from logivations/update_with_min
eac506a 
add function updateWithMin
@jplapp
spin without costmap
1da25a3
@jplapp
Merge pull request #70 from logivations/spin_without_costmap
d8b26e1 
spin without costmap
@jplapp
zero copy pub with transient local
9c88ab7
@AnastasiaPittel
use node name
204c07c
@papertowel123
backport
e831927
@VladyslavHrynchak200204
Fix Flickering visualization
76f1d14
@VladyslavHrynchak200204
Refactoring Costmap2DPublisher and Costmap2DROS
00abe59
@VladyslavHrynchak200204
Refactoring costmap_2d_ros.cpp
4cd39fd
@VladyslavHrynchak200204
Change map_vis_y into map_vis_z
9524349
@VladyslavHrynchak200204
Update costmap_2d_ros.cpp
eb850ab
@VladyslavHrynchak200204
Merge pull request #72 from logivations/AMRNAV-6557_Fix_flickering_vi…
4605fc6 
…sualization

Amrnav 6557 fix flickering visualization
@papertowel123
Merge pull request #71 from logivations/backport_velocity_polygon
b87b8ec 
Backport velocity polygons
@papertowel123
Merge pull request #73 from logivations/support_dynamic_name_bt_navig…
60bc8a4 
…ator

Support dynamic name bt navigator
@papertowel123 papertowel123 requested a review from jplapp as a code owner July 24, 2024 14:09
@papertowel123
Copy link
Author

papertowel123 commented Jul 24, 2024

aligning everything to continue use iron branch

@papertowel123
add param read
c2c182f 
Signed-off-by: maksymdidukh <[email protected]>
@papertowel123
.
4847edc 
Signed-off-by: maksymdidukh <[email protected]>
@papertowel123
add tests
44a10f3 
Signed-off-by: maksymdidukh <[email protected]>
papertowel123 and others added 5 commits July 30, 2024 15:38
@papertowel123
fix incorrect param use
29d4fcd 
Signed-off-by: maksymdidukh <[email protected]>
@papertowel123
uncrustify
f022ddc 
Signed-off-by: maksymdidukh <[email protected]>
@papertowel123
add test draft
eed24f9 
Signed-off-by: maksymdidukh <[email protected]>
@papertowel123
comment unfinished tests
dc160e1
@papertowel123
Merge pull request #75 from logivations/improve_velocity_polygon_params
e137193 
improve param read in velocity polygon
@papertowel123 papertowel123 changed the title Transient local publish merge transient_local_publish into iron Jul 31, 2024
@papertowel123 papertowel123 self-assigned this Jul 31, 2024
jplapp
jplapp approved these changes Aug 12, 2024
View reviewed changes
Copy link

@jplapp jplapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this ready?

@papertowel123
Copy link
Author

papertowel123 commented Aug 15, 2024 •
edited
Loading

is this ready?

yes, I think so, drone tests are passing on this branch: https://drone.logivations.com/logivations/deep_cv/43118

@papertowel123 papertowel123 merged commit e296ac9 into iron Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jplapp jplapp jplapp approved these changes

Assignees

@papertowel123 papertowel123

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@papertowel123 @jplapp @AnastasiaPittel @VladyslavHrynchak200204