-
Notifications
You must be signed in to change notification settings - Fork 132
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
Feature/raw4 #778
Feature/raw4 #778
Conversation
jabbenseth
commented
Jun 3, 2019
- fixes #
- new source code files added (proper APACHE license header must be used)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
supersedes #777?
see #777 (review)
will further review once it's funal
not at all
|
|
could be checked/guaranteed by adding a temporary commit with the required feature branches/remotes in the |
@ipa-jba do the same with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still has potential for optimization
<arg name="sim" default="false"/> | ||
|
||
<arg name="driver_yaml" value="$(arg pkg_hardware_config)/robots/raw4/config/raw4_base_driver.yaml"/> | ||
<arg name="controller_yaml" value="$(arg pkg_hardware_config)/robots/raw4/config/raw4_base_controller.yaml"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are those files different for raw3 vs. raw4?
if not: stick with one single raw_base.launch
<remap from="scan" to="scan_raw"/> | ||
<!-- <remap from="extended_laser_scan" to="laser_$(arg name)/extended_laser_scan"/> | ||
<remap from="output_paths" to="laser_$(arg name)/output_paths"/> | ||
<remap from="raw_data" to="laser_$(arg name)/raw_data" /> --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indentation?
cob_bringup/robots/raw4-0.launch
Outdated
<arg name="env-script" default="$(find cob_bringup)/env.sh"/> | ||
|
||
<include file="$(find cob_bringup)/robots/raw4-0.xml"> | ||
<!--arg name="pc1" value="i60sr2"/--> <!-- todo --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix todo!
|
||
<include file="$(find cob_bringup)/tools/teleop_common.xml" pass_all_args="true"/> | ||
|
||
<node ns="teleop" pkg="teleop_twist_joy" type="teleop_node" name="$(anon teleop_node)" cwd="node" respawn="false" output="screen"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does this node do?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it replaces cob_teleop since the cob version imply a load of dependencies not available on melodic (cob_script_server which relies on cob_light...)
since raw4 are melodic robots we use the ros provided, released teleop node
##Wheel 3 | ||
br_caster_rotation_joint: | ||
motor_layer: | ||
homing_event: 17 # -> digin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why does this wheel have a different homing_event?
motor_layer: | ||
homing_event: 19 # -> digin | ||
homing_speed: -5093 # -1.0 rs/s | ||
homing_offset: 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the base seems uncalibrated!
@@ -0,0 +1,33 @@ | |||
# !!! copy pasted, needs to be adjusted !!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix todo?
@@ -0,0 +1,33 @@ | |||
# !!! copy pasted, needs to be adjusted !!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix todo?
@@ -0,0 +1 @@ | |||
../../raw4/config/pc_monitor_raw4-pc1.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you could use symlinks to raw4/config for a lot more raw4-0/config files!
superseded by #798 |