forked from ros/ros_comm
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Synchronize with melodic-devel. #2
Open
peci1
wants to merge
136
commits into
cwecht:parametrize_timeouts
Choose a base branch
from
peci1:parametrize_timeouts
base: parametrize_timeouts
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lback may use the publisher (ros#1669)
* publishtest: use AnyMsg in publishtest * fix concerns
* rostest: fix flaky hztests * add retry to all hztests * fix concerns * fix more wrong retry-attributes
) * fix ros#1474 move bag encryption plugins into separate library This setup is required by the class_loader, which is part of the pluginlib, see http://wiki.ros.org/class_loader#Caution_of_Linking_Directly_Against_Plugin_Libraries * fix unit tests of aes_encryptor
…etParam (ros#1439) * test_roscpp/params/added getParamCachedSetParamLoop * rosmaster: set_param: the not update the caller! * rosmaster: set_param: do not update the caller more fine grained * /rosmaster/paramserver/compute_params_update, apply filter only if caller_id_to_ignore is not None * /test_rospy/talker: set publishers queue_size to supress warning * /test_rospy/sub_to_multple_pubs: moved listener up to avoid warnings * refactor for readability * pep8
* test_rosbag/test_bag.py: test, if rosbag exception can be pickled * rosbag/bag.py: rosbag exceptions can now be unpickled * pep8
* fix broken changes. (ros#55) Fix broken changes. * add comment, include rosbag_storage first * remove unhelpful comment
… possible (ros#1698) * rosgraph/network: use urlparse for parsing the port, this makes ipv6 possible * remove unnecessary assert and comment
* Resolve memory leak. Delete g_rosout_appender explicitly instead of assigning it to NULL. Follow deletion with NULL assignment. * Deregister g_rosout_appender * revert unrelated whitespace change * Update init.cpp * Increment version number. * space in comments * Merge * Revert "Increment version number." This reverts commit 795c8fd. * Add newer rosconsole dependencies * Update rosconsole dependencies in package.xml * Sync with upstream completely * Add changes * Remove deregister function, since it is already done in shutdown(). * Remove unnecessary .catkin_workspace file.
There is no s variable in scope - and we clearly wanna display reg_type
* Switch to yaml.safe_load(_all) to prevent YAMLLoadWarning * Change all usages of yaml.load to yaml.safe_load * Extend PyYAML's SafeLoader and use it with `yaml.load` Also added convenience functions for using this loader for reuse in `roslaunch` * fix typo in rosparam.yaml_load_all * Modify Loader and SafeLoader in yaml module directly * Revert whitespace change * Revert unrelated change to import through global variable construction
The missing comma will implicitly concatenate the string "FATAL" and "is_shutdown" together
Signed-off-by: Daniel Wang <daniel.wang@canonical.com>
…ve correct type now (ros#1780)
fix ros#1753 Signed-off-by: artivis <jeremie.deray@canonical.com>
* roscpp/transport_udp: zero-initialize sockaddr_in object * rostcp/transport_udp: zero-initialize sockaddr_in members
…#1704) connections properly
* Allow empty machine tag. Fixes ros#274. * Fix tests for machine arg in node tag. * Remove obsolete test. * Remove obsolete test
* Add default ROS_MASTER_URI * roscpp: added getDefaultMasterUri() * moved DEFAULT_MASTER_PORT and DEFAULT_MASTER_URI from rospy to rosgraph to make them usable in get_master_uri * removed not needed try-catch-block in get_master_uri * style of touched lines * style of touched lines * style of touched lines Co-authored-by: Jochen Sprickerhof <github@jochen.sprickerhof.de> Co-authored-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
* add assertion to narrow down error case * update test to ignore messages which are insufficient
…ness (ros#1897) * increase time limit of publishtest.test to reduce flakyness * increase time limit of advertisetest.test to reduce flakyness
Signed-off-by: Miaofei <miaofei@amazon.com>
macOS was incorrectly attempting to use epoll/poll instead of kqueue, occassionally generating errors such as RuntimeError: concurrent poll() invocation and breaking python nodes. Co-authored-by: Steve Nogar <snogar@gmail.com>
Prevent python nodes from spamming the terminal every time a node is quit. Noted in Python 3 running on macOS. Co-authored-by: Steve Nogar <snogar@gmail.com>
changes between 1.15.4 (Noetic) and 1.14.4 for backporting into Melodic
peci1
force-pushed
the
parametrize_timeouts
branch
from
April 20, 2020 23:39
498eabc
to
212f2c3
Compare
peci1
force-pushed
the
parametrize_timeouts
branch
from
April 21, 2020 01:12
bbd0862
to
f1774f2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I think we could move on with ros#1670, couldn't we?