search

RobotnikAutomation / summit_xl_common

URDF description of the Summit XL and Summit XL HL, platform messages and other files for simulation.
BSD 2-Clause "Simplified" License
72 stars 47 forks source link

Porting over to Melodic #7

Closed acxz closed 5 years ago

acxz commented 5 years ago

Hello, I am wondering if this repo will be packaged as a Melodic package, and if so when? I am interested in getting this code to work in Melodic and I would love to use packages instead of compiling the source code.

RomanRobotnik commented 5 years ago

Hi,

The migration to Melodic is planned to be done during this year. I give you an exact date yet, sorry!

acxz commented 5 years ago

Thank you for responding! Can there be a branch made for melodic? As I am working on the source code it would be great if there is a branch I can submit issues and PRs to.

JackBonnellDevelopment commented 5 years ago

Hi i am trying to run the summit xl on melodic and i am only getting these errors since upgrading to ubuntu 16.4 and melodic. The velocity is set to max and there seems to be no steering the image below shows the errors that are being shown in the main console and the .../robotnik_base_hw/status topic. logs

jgomezgadea commented 5 years ago

Hi,

I'm not sure if I've understood your question correctly, but I think that kinetic is not supported in Ubuntu 18.04. Here you will find the different versions along with their Ubuntu versions.

The terminal has no error messages, there are info and warn messages (and it's working correctly).

The steering problem seems a problem with the configuration of your robot (on ~/robot_params.env). Make sure you have set the params correctly according with your robot. If you don't know the configuration of your model, contact with support@robotnik.es giving us the serial number of your robot.

acxz commented 5 years ago

I believe this issue has been incorrectly closed since this package has not been ported over to melodic.

jgomezgadea commented 5 years ago

I believe this issue has been incorrectly closed since this package has not been ported over to melodic.

Hi @acxz , I think the question asked on this issue has been answered. If you have another question related with this, do not hesitate to ask.

As soon as we could, we will create a new branch for melodic.

acxz commented 5 years ago

My question has been addressed with your last comment. Although I believe this issue should remain open until that branch is made, for the sake of project trace-down, but I understand if you want to close this issue.