Open xmfcx opened 1 year ago
@kenji-miyake @mitsudome-r @esteve maybe we can also discuss about whether we should build these packages as part of the CI or not.
Or discuss whether we should do the proposal on this issue at all.
From these 3, I think velodyne is the more essential one but I'm not sure about what to do with the others. Any thought is appreciated.
@xmfcx I think pacmod could be moved out easily, but as you say Velodyne is essential to our architecture, so I'm not sure if it should be moved out. The Tamagawa IMU driver doesn't seem to be used much, though. Only two packages depend on it: sample_sensor_kit_launch
and awsim_sensor_kit_launch
@xmfcx I'm sorry for joining late. Explaining the background, these are from TIER IV's reference vehicle/sensor_kit.
I think splitting repos or just removing some is okay, but there are several things to be discussed:
Do you really want to split drivers? Or do you want to split the entire sensor components? I'm not sure which is better.
Since not every vehicle will use all the drivers, it's better for them to have a separate driver.repos file.
Seeing this purpose, I feel like you want to split the sensor components that aren't related to your vehicle.
This way when people fork autoware, it's easier for them to disable these repositories.
For this, I think users can do it with the current configuration because they are under the sensor_component
section. :thinking:
Could you please show me some concrete examples?
We might need to fix some dependencies before splitting them out.
Anyway, I'd like to clarify the background and issues more. Honestly, I feel that the current proposal might be a bit short-sighted. So could you share your troubles in more detail with me?
@xmfcx Could you share more detail about the purpose of this issue?
If latter is the case, I think we can just remove Tamagawa-imu and pacmod-interface drivers since they are not used in tutorials anyways.
So, I have thought about this more generally and here are the things I'd like to achieve:
For this, I will separate out the following:
pacmod_interface
tamagawa_imu_driver
into a different .repos
file called extra-packages.repos
.
Will modify the CI to pull them along with the others, build and test as usual.
The users will download the usual autoware.repos
without these packages.
If they want to, they can download it with the extra-packages.repos
.
In the future, we can think of similar non-essential external packages to this list.
Do you think this is reasonable? @esteve @mitsudome-r ?
@xmfcx I agree with you, the drivers are quite specific to the vehicle where Autoware is running, so it'd be better to move them to a separate .repos
file and let users use whatever driver they need for their vehicle.
Checklist
Description
This issue is about moving the driver packages to a separate
driver.repos
file from autoware.repos file.Purpose
Since not every vehicle will use all the drivers, it's better for them to have a separate
driver.repos
file.This way when people fork autoware, it's easier for them to disable these repositories.
Possible approaches
We should simply separate the driver repositories.
But we will probably need to update the CI and the documentation for installation.
Current drivers:
Definition of done
pacmod_interface
is moved outvelodyne_vls
is moved outtamagawa_imu_driver
is moved out