AutonomousVehicleControlBeginnersGuide icon indicating copy to clipboard operation
AutonomousVehicleControlBeginnersGuide copied to clipboard

Packaging Support

Open egeKeyvan opened this issue 1 year ago • 4 comments

Thanks for the great work with this library. I have a feature request though.

Currently, in order to find the other files in the project, you add them to the path manually. This is not a good practice in general and leads to several problems. One of them is I am not able to get any Language Server completions since LSP can't recognize the path additions. This makes navigating the code much harder. Also, with this hardcoded addition, the library cannot be installed and used with other libraries. For example, if I had a different mapper than NDT, i can't use it with the existing code easily without making additions to your library as well which would make it difficult to keep up with the newer Github releases.

My suggestion is to package the project using pyproject.toml so that library users can install this to their machine and get full LSP support and flexibility in adding additional stuff.

One more suggestion is also use a linter and formatter such as ruff. It would greatly help in organizing the imports, enforce good code and provide better readibility.

If you are open to it, I can add pyproject support with tooling.

egeKeyvan avatar Dec 22 '23 12:12 egeKeyvan

Hi, @egeKeyvan. Thank you for suggesting the good improvement idea!!
This is the first opened issue for my project, I'm so glad!!

I want to ask you to add pyproject support and send me a pull request, but a document about how to contribute to this project has not been prepared well yet. So, I will add by myself this time.

I will let you know here after I could add.
Thank you for your suggestion again!!

ShisatoYano avatar Dec 23 '23 01:12 ShisatoYano

Great, thanks for the effort on improving the library! Please let me know if there is anything you need help with. Also, please consider having a publication list for referencing in case this work is useful in academic context

egeKeyvan avatar Dec 25 '23 12:12 egeKeyvan

Hi, Do you have any updates with this ?

egeKeyvan avatar Jan 11 '24 09:01 egeKeyvan

I've been doing but not completed yet because I have been so busy these days.
Please for a while more.

ShisatoYano avatar Jan 11 '24 14:01 ShisatoYano