Python Scripts from Zorn and Ronecker SS17#1
Conversation
|
|
||
| class Controler: | ||
| def __init__(self, image_topic='torcs_ros/pov_image',control_topic='/torcs_ros/ctrl_cmd', sensors_topic='/torcs_ros/sensors_state'): | ||
| self.model_path = '/home/max/Schreibtisch/weights_epoch_99.h5' #load neural net at path |
There was a problem hiding this comment.
@B-Zorn please make the model path a parameter instead of hard coding it here! Will you upload an example of one trained model to the LRZ shared drive or add it to this PR (depending on size of the file)?
There was a problem hiding this comment.
I'll upload a trained model to the LRZ drive
There was a problem hiding this comment.
Awesome, thanks for your effort 😃
| self.sleep_rate = 0.2 #variable used to limit data to every x seconds | ||
| self.extension ="ben" #define user prefix for file name | ||
| self.find_dir() #find current working directory | ||
| self.evalFactor = 1.5 #factor used for eval |
|
@B-Zorn looks good in general. Would it be possible to put the data-logger in a separate ROS-package, as its functionality is kind of separated from the controller and the network training? Furthermore, please have a look at the folder-structure of the other packages in the repository to make the purpose of individual files a bit more self-explanatory. Thanks :) |
|
@fmirus: I'd argue the data-logger's functionality is very similar as it is identical to basically all other ROS nodes in the package. It subscribes, (synchronizes and saves data) and then republishes said synchronized data if needed. |
… Added YAML parameters in datalogger
|
@B-Zorn Agree, that the data-logger publishes/subscribes data in a similar way as the other nodes, however the functionality of the node itself is different to what e.g. torcs_ros_drive_ctrl does (logging vs controlling the vehicle). That's why I would like to put it into a separate ROS-package within the same repository. Same holds true, for your DNN-based driver node. |
|
@fmirus |
minor fix to avoid watchdog crash
2 participants
No description provided.