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

Humble corrections #915

Closed
wants to merge 23 commits into from
Closed

Humble corrections #915

wants to merge 23 commits into from

Conversation

nidhicj
Copy link

@nidhicj nidhicj commented Jun 20, 2024

Added a fdlimit = int(line[len(string_to_find):].strip()) in ros2cli/ros2cli/node/daemon.py replacing use of "removeprefix" as it is supported only from python3.9

mergify bot and others added 23 commits July 25, 2022 15:43
…ros2#733)

Signed-off-by: Tomoya Fujita <[email protected]>
(cherry picked from commit c7d3204)

Co-authored-by: Tomoya Fujita <[email protected]>
…eader field (ros2#751)

* Expand auto to the current time when passed to a Header field

Signed-off-by: Esteve Fernandez <[email protected]>

* Make sure that the stamp field is updated every time the message is published

Signed-off-by: Esteve Fernandez <[email protected]>

* Added support for passing 'now' as a value to a 'builtin_interfaces.msg.Time'. Added comments and expand docstring

Signed-off-by: Esteve Fernandez <[email protected]>

* Only update timestamp fields that have been passed 'now' as a value

Signed-off-by: Esteve Fernandez <[email protected]>

* Fix lint errors

Signed-off-by: Esteve Fernandez <[email protected]>

* Added tests

Signed-off-by: Esteve Fernandez <[email protected]>

* Remove node argument from set_message_fields_expanded

Signed-off-by: Esteve Fernandez <[email protected]>

* Fix flake8 errors

Signed-off-by: Esteve Fernandez <[email protected]>

Signed-off-by: Esteve Fernandez <[email protected]>
…os2#757)

Signed-off-by: Chris Lalancette <[email protected]>
(cherry picked from commit e9789d4)

Co-authored-by: Chris Lalancette <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
* Add support use_sim_time for ros2 topic hz/bw/pub.

Signed-off-by: Lei Liu <[email protected]>

* Add testcase.

Signed-off-by: Lei Liu <[email protected]>

* Update code.

Signed-off-by: Lei Liu <[email protected]>

* Update code again.

Signed-off-by: Lei Liu <[email protected]>

* Add warning log.

Signed-off-by: Lei Liu <[email protected]>

* Revert STEADY_TIME.

Signed-off-by: Lei Liu <[email protected]>

* Fix flake8 warnings.

Signed-off-by: Lei Liu <[email protected]>

* Update code.

Signed-off-by: Lei Liu <[email protected]>

Signed-off-by: Lei Liu <[email protected]>
Signed-off-by: Tomoya Fujita <[email protected]>

Signed-off-by: Lei Liu <[email protected]>
Signed-off-by: Tomoya Fujita <[email protected]>
Co-authored-by: Lei Liu <[email protected]>
…#784)

address flaky test: ros2#772

Signed-off-by: Tomoya Fujita <[email protected]>
(cherry picked from commit 615efe6)

Co-authored-by: Tomoya Fujita <[email protected]>
* NewtorkAwareNode: Fix issue of using old DirectNode in method calls

Signed-off-by: Ivan Santiago Paunovic <[email protected]>

* NetworkAwareNode: Check all interfaces, not only the gateways

Signed-off-by: Ivan Santiago Paunovic <[email protected]>

Signed-off-by: Ivan Santiago Paunovic <[email protected]>
(cherry picked from commit 70d4991)

Co-authored-by: Ivan Santiago Paunovic <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
…os2#798)

Signed-off-by: Tomoya Fujita <[email protected]>
(cherry picked from commit e2adeac)

Co-authored-by: Tomoya Fujita <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
…os2#813) (ros2#815)

Otherwise, the 'use_sim_time' parameter we are setting here is
completely ignored.

Signed-off-by: Chris Lalancette <[email protected]>
(cherry picked from commit 08fdf98)

Co-authored-by: Chris Lalancette <[email protected]>
Signed-off-by: ijnek <[email protected]>
(cherry picked from commit ad0c2a8)

Co-authored-by: Kenji Brameld <[email protected]>
"""Dump the parameters of a node to a yaml file.""" -> """Show all of the parameters of a node in a YAML file format."""

Based on the documentation at
https://docs.ros.org/en/rolling/How-To-Guides/Using-ros2-param.html#ros2-param-dump

Signed-off-by: Murilo M. Marinho <[email protected]>
(cherry picked from commit ef5d811)

Co-authored-by: Murilo M Marinho <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
Signed-off-by: Tomoya Fujita <[email protected]>
(cherry picked from commit 0084d12)

Co-authored-by: Tomoya Fujita <[email protected]>
…pletion (ros2#852) (ros2#873)

* make file handles not inheritable before spawn the daemon

Signed-off-by: Chen Lihui <[email protected]>
Co-authored-by: Chris Lalancette <[email protected]>
(cherry picked from commit d32d820)

Co-authored-by: Chen Lihui <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
On Python 3.6 (in RHEL-8), signal.valid_signals()
does not exist.  Only use it if it exists.

Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Scott K Logan <[email protected]>
Signed-off-by: Tomoya Fujita <[email protected]>
(cherry picked from commit 305ef76)

Co-authored-by: Tomoya Fujita <[email protected]>
Signed-off-by: Audrow Nash <[email protected]>
…ros2#888) (ros2#908)

* ros2cli.node.daemon : try getting fdsize from /proc for open fd limit

* - add with-block; remove linux check

* - use removeprefix

* - replace filter with loop

Signed-off-by: akssri <[email protected]>
(cherry picked from commit 64d216c)

Co-authored-by: akssri-sony <[email protected]>
@clalancette
Copy link
Contributor

This is a duplicate of #912

@nidhicj nidhicj deleted the humble_corrections branch June 20, 2024 12:44
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.

6 participants