ROSIntegration icon indicating copy to clipboard operation
ROSIntegration copied to clipboard

repo name confusing

Open daniel86 opened this issue 8 years ago • 4 comments

it should be something like "unreal_ros_integration" instead.

Note also the convention writing the repo names lowercase with underscore.

daniel86 avatar Jan 09 '18 08:01 daniel86

What convention do you mean? I personally would like to omit "unreal" from the name because it would be redundant as a plugin that only works with unreal.

Hemofektik avatar Mar 20 '18 13:03 Hemofektik

the rather implicit conventions can be derived by looking at how repositories are named in the code-iai group -- which is lower case with underscore.

I would add the hint that it is unreal related in the name of the repository to avoid confusion about what is integrated with ROS -- the IAI is integrating lots of stuff with ROS ;)

daniel86 avatar Mar 20 '18 14:03 daniel86

The current naming is aligned with the Unreal Naming Convention which is nice, but I understand your concern as well. It's probably a matter of opinion in the end.

Hemofektik avatar Mar 20 '18 14:03 Hemofektik

Well why don't you create then a "UnrealRobots" group or so? Then everything in this group can be named according to Unreal conventions.

And it's not only opinion but more consistency

daniel86 avatar Mar 21 '18 11:03 daniel86