description dev #1
Merged
Conversation
This file contains hidden or 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
Update from upstream repo
fixes problem that no messages are published if robot program is not running
Co-Authored-By: Felix Exner <[email protected]>
This launchfile was created for local testing in the past and slipped through.
Use github actions for CI instead of Travis
…iversalRobots#115) * Improve the quick start guide to make the start order more clear. * Added a couple of suggested improvements Thanks @gavanderhoorn * Made clear that we are extracting calibration, not calibrating the robot.
Before, the main badge could get to failure when there was a PR coming in that failed.
…entation Use section commands for each individual topic/service/parameter url
set motion_possible to true only of robot can be actually moved
…tch-1 Use badge only from pushes
Add integration tests for automated testing
Use a parameter to set the wrench name This name will be picked up by the `force_torque_sensor_controller` in order to name the respective topic. Co-authored-by: carebare47 <[email protected]>
* Added a service to setup the active payload
Before, the default Docker network in the range 172.17.0.0/16 was used. Since a specific IP cannot be chosen/guaranteed within this range, a network is now created with range 192.168.0.0/16, which allows for assigning specific IPs to the containers. Co-authored-by: Emil Vincent Ancker <[email protected]>
modified hardware interface to look for joints parameter under the robot_hw node handle
The trajectory test seems to not work anymore since a couple of weeks. Running those locally (also with a ursim running inside a docker container) works perfectly fine, but running it inside the github action not. As this is blocking many merges currently, I suggest to disable this temporarily while opening an issue to fix it.
* Added explanation of scaled trajectory controller * Apply suggestions from code review Thanks @gavanderhoorn for this in-depth review! Co-authored-by: G.A. vd. Hoorn <[email protected]> * Added another suggestion I missed Co-authored-by: G.A. vd. Hoorn <[email protected]> * Clarified where the speed_scaling value comes from. * Format ros_control correctly * Rewrote bullet point about non-moving robot * Replace time frame with control cycle Co-authored-by: G.A. vd. Hoorn <[email protected]>
* Updated externalcontrol to v1.0.3 * Updated externalcontrol to v1.0.4
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.
ref: UniversalRobots#97
update
mojin-develbranch to support e-series