Skip to content

Commit

Permalink
load environment variable for setting robot in joint-state-compressor.l
Browse files Browse the repository at this point in the history
  • Loading branch information
mmurooka authored and eisoku9618 committed Apr 15, 2015
1 parent decca9f commit 39089ec
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions jsk_network_tools/euslisp/joint-state-compressor.l
Expand Up @@ -21,9 +21,9 @@
(string-upcase *message-package*))))
;; load robot model

(setq *robot-name* (ros::get-param "~robot" "pr2"))
(setq *robot-name* (string-upcase (unix::getenv "ROBOT")))

This comment has been minimized.

Copy link
@k-okada

k-okada Apr 25, 2015

Member

why you use "ROBOT" env only for compression and not in decompression program?
https://github.com/jsk-ros-pkg/jsk_common/blob/master/jsk_network_tools/euslisp/joint-state-decompressor.l#L24

(unless *robot-name*
(error "Please specify ~~robot_name"))
(error "Please specify environment variable \"ROBOT\""))
;; load robot files... it's not so good
(load-robot-model *robot-name*)
(setq *robot* (make-robot-model-from-name *robot-name*))
Expand Down

0 comments on commit 39089ec

Please sign in to comment.