refactor(glog_component): prefix package and namespace with autoware

[esteve]

Description

prefix package and namespace with autoware

Related links

Parent Issue:

• Reorganize the autoware.universe/common packages autoware#5077
• Prefix packages with autoware_ prefix and keep code in autoware namespace autoware#4569

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

[github-actions]

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

• You've checked our contribution guidelines.
• Your PR follows our pull request guidelines.
• All required CI checks pass before marking the PR ready for review.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 29.46%. Comparing base (ed133eb) to head (1200ad5).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...mon/autoware_glog_component/src/glog_component.cpp	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9302      +/-   ##
==========================================
- Coverage   29.51%   29.46%   -0.05%     
==========================================
  Files        1446     1455       +9     
  Lines      108518   108593      +75     
  Branches    41396    41400       +4     
==========================================
- Hits        32025    31999      -26     
- Misses      73372    73476     +104     
+ Partials     3121     3118       -3     

Flag	Coverage Δ		*Carryforward flag
differential	9.78% <0.00%> (?)
total	29.48% <ø> (-0.03%)	⬇️	Carriedforward from ed133eb

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[xmfcx]

Sorry I pasted it from the other place, the table refers to glog_component. Edited the comment.

I've renamed the title to refactor because it is renaming the package name in source files and file names themselves.

Since there are code changes, we cannot use chore to refer to these prefixing PRs.

References:

• https://gist.github.com/joshbuchea/6f47e86d2510bce28f8e7f42ae84c716
• https://stackoverflow.com/a/26944812/4520129

[esteve]

I agree with refactor, I've used it in the past, but our pr-label check was failing.

[esteve]

@xmfcx I've submitted the following PRs to address your feedback:

• refactor(glog_component): prefix package and namespace with autoware autoware_tools#158
• refactor(glog_component): prefix package and namespace with autoware awsim_labs_sensor_kit_launch#10
• refactor(glog_component): prefix package and namespace with autoware sample_sensor_kit_launch#110
• refactor(glog_component): prefix package and namespace with autoware single_lidar_sensor_kit_launch#4
• refactor(glog_component): prefix package and namespace with autoware autoware_launch#1245

[isamu-takagi]

When I tested this file, the class could not be found, so I think the following changes are necessary. The same goes for the launch file.

<?xml version="1.0"?>
<launch>
  <node_container pkg="rclcpp_components" exec="component_container" name="glog_container" namespace="">
    <composable_node pkg="autoware_glog_component" plugin="GlogComponent" name="glog_component"/>
  </node_container>
</launch>

[esteve]

@isamu-takagi thanks, I've applied your suggestions.

[xmfcx]

I was able to drive autonomously with AWSIM-Labs with the latest autoware nightly repos, with all the sub-PRs rebased and checked out to main. ✅

Thanks for your efforts!

[xmfcx]

Also manually checked the usage of glog_component throughout the repository and checking them visually, they all seemed to have been prefixed correctly ✅