PepperlFuchs / pf_lidar_ros_driver

ROS driver for Pepperl+Fuchs R2000 and R2300 laser scanners
https://www.pepperl-fuchs.com/global/en/23097.htm
Apache License 2.0
39 stars 38 forks source link

Maintenance status driver #62

Closed JaccovdS closed 3 years ago

JaccovdS commented 3 years ago

I wanted to share with you that we're not that happy with the maintenance status of the driver/repo given our experience over the last year. Here are some examples why:

  1. Official release took a very long time, first mention was May 2020 actual date of the first tag (not even ROS release) was November 2020. Reference: https://github.com/PepperlFuchs/pf_lidar_ros_driver/issues/12 https://github.com/PepperlFuchs/pf_lidar_ros_driver/issues/34
  2. No Noetic support, also takes quite some time. Reference: https://github.com/PepperlFuchs/pf_lidar_ros_driver/issues/28
  3. Meaningfull comments and PR are closed (due to insufficient knowledge?). Reference: https://github.com/PepperlFuchs/pf_lidar_ros_driver/issues/37 https://github.com/PepperlFuchs/pf_lidar_ros_driver/pull/36#issuecomment-736545604
  4. Issues stay open for a long time. Reference: https://github.com/PepperlFuchs/pf_lidar_ros_driver/issues/38

In general what we lack is response within a reasonable timespan and constructive discussions. If you truly want to have an open-source driver we would expect a more active approach in maintaining the driver.

What is your opinion on this and what are your plans for the future?

@PepperlFuchs @ipa-hsd

hsd-dev commented 3 years ago

@JaccovdS sorry for your bad experience about the driver. I will try to explain your concerns below:

  1. The development of the driver began before the new device (R2300) was officially released. Once the device was released, the customers tried out the driver and reported the bugs (performance issues, crashes, etc) which were important and needed immediate attention than the issues linked above. The performance related issues have been promptly fixed (please report if you are still facing any issues). It did not make sense to release the driver with the main issues still unresolved.
  2. To make the driver robust, we concentrated on only one distro (Melodic). Now that the main issues have been solved and the driver has been released for Melodic, we are starting with Noetic support.
  3. Latency, even though an important aspect, was not a blocker compared to performance, crashes and other bugs (https://github.com/PepperlFuchs/pf_lidar_ros_driver/issues/47). Latency and re-connectivity will be addressed in the second phase of the project.
  4. Please see above.

Quite some work did happen in the last year (https://github.com/PepperlFuchs/pf_lidar_ros_driver/commits/main) including porting the continuous integration from TravisCI to Github Actions and will continue to happen in the coming months.

JaccovdS commented 3 years ago

@ipa-hsd I'm in no doubt that a lot of useful work has been done. That's also no point of discussion.

My main point is that you now make your source code publicly available on this platform but you don't seem to be open to open-source development. I.e. accepting / discussing useful PR's, listening to comments / requests / needs, being clear about milestones / timelines / goals. In my opinion open-source development comes with responsibility towards the community.

We as a company are willing to contribute but based on the past experiences we are faster and better when we take this source and continue on our own. I think this is something you want to prevent as maintainer of an open-source project.

hsd-dev commented 3 years ago

That is a very valid point @JaccovdS. We will discuss this with @PepperlFuchs and improve the contributions from the community in the future.

JaccovdS commented 3 years ago

@ipa-hsd happy to hear. Can you update us on the outcome?

JaccovdS commented 3 years ago

@ipa-hsd any news?

JaccovdS commented 3 years ago

@ipa-hsd @PepperlFuchs ping

JaccovdS commented 3 years ago

@ipa-hsd @PepperlFuchs I think the lack of feedback in this issue proves my point. I will close the issue.