Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(raw_vehicle_cmd_converter): add contents of vehicle_adaptor #9759

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

masayukiaino
Copy link
Contributor

Description

Add contents of vehicle adaptor

Related links

Parent Issue:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

Copy link

github-actions bot commented Dec 24, 2024

Thank you for contributing to the Autoware project!

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

Please ensure:

@github-actions github-actions bot added type:documentation Creating or refining documentation. (auto-assigned) component:vehicle Vehicle-specific implementations, drivers, packages. (auto-assigned) labels Dec 24, 2024
@kosuke55 kosuke55 self-assigned this Dec 24, 2024
@kosuke55 kosuke55 force-pushed the tmp/vehicle_adaptor branch from 1a15cbb to 5d1aa61 Compare December 24, 2024 06:21
@kosuke55 kosuke55 changed the title feat(raw_vehicle_cmd_converter): update vehicle adaptor feat(raw_vehicle_cmd_converter) Add contents of vehicle_adaptor Dec 24, 2024
@kosuke55 kosuke55 changed the title feat(raw_vehicle_cmd_converter) Add contents of vehicle_adaptor feat(raw_vehicle_cmd_converter) add contents of vehicle_adaptor Dec 24, 2024
@kosuke55 kosuke55 added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Dec 24, 2024
@kosuke55 kosuke55 changed the title feat(raw_vehicle_cmd_converter) add contents of vehicle_adaptor feat(raw_vehicle_cmd_converter): add contents of vehicle_adaptor Dec 25, 2024
@kosuke55 kosuke55 force-pushed the tmp/vehicle_adaptor branch from 789b42f to 2ac7aa3 Compare December 25, 2024 10:51
@kosuke55
Copy link
Contributor

kosuke55 commented Dec 25, 2024

todo

  • update README
  • fix CI
    • build
    • spell check
    • ...
  • add const
  • use hpp
  • remove unnecessary functions
  • remove unnecessary files or add descriptions for them

Signed-off-by: kosuke55 <[email protected]>
@kosuke55
Copy link
Contributor

I did not check the details but fixed build error anyway e6b4f27

../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp: In member function ‘autoware_control_msgs::msg::Control autoware::raw_vehicle_cmd_converter::VehicleAdaptor::compensate(const Control&, const Odometry&, const AccelWithCovarianceStamped&, double, const OperationModeState&, const ControlHorizon&)’:
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:41:8: error: ‘proxima_vehicle_adaptor_’ was not declared in this scope
   41 |   if (!proxima_vehicle_adaptor_.use_vehicle_adaptor_) {
      |        ^~~~~~~~~~~~~~~~~~~~~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:46:8: error: ‘initialized_’ was not declared in this scope
   46 |   if (!initialized_) {
      |        ^~~~~~~~~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:47:5: error: ‘proxima_vehicle_adaptor_’ was not declared in this scope
   47 |     proxima_vehicle_adaptor_.clear_NN_params();
      |     ^~~~~~~~~~~~~~~~~~~~~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:61:3: error: ‘Eigen’ has not been declared
   61 |   Eigen::VectorXd states(6);
      |   ^~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:62:48: error: ‘yaw_prev_’ was not declared in this scope
   62 |   const double yaw = get_current_yaw(odometry, yaw_prev_);
      |                                                ^~~~~~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:63:3: error: ‘states’ was not declared in this scope; did you mean ‘static’?
   63 |   states[0] = odometry.pose.pose.position.x;
      |   ^~~~~~
      |   static
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:72:5: error: ‘proxima_vehicle_adaptor_’ was not declared in this scope
   72 |     proxima_vehicle_adaptor_.send_initialized_flag();
      |     ^~~~~~~~~~~~~~~~~~~~~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:76:7: error: ‘proxima_vehicle_adaptor_’ was not declared in this scope
   76 |   if (proxima_vehicle_adaptor_.use_controller_steer_input_schedule_){
      |       ^~~~~~~~~~~~~~~~~~~~~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:83:3: error: ‘Eigen’ has not been declared
   83 |   Eigen::Vector2d vehicle_adaptor_control_cmd =
      |   ^~~~~
../../src/autoware/universe/vehicle/autoware_raw_vehicle_cmd_converter/src/vehicle_adaptor/vehicle_adaptor.cpp:87:50: error: ‘vehicle_adaptor_control_cmd’ was not declared in this scope
   87 |   output_control_cmd.longitudinal.acceleration = vehicle_adaptor_control_cmd[0];
      |                                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~
gmake[2]: *** [CMakeFiles/vehicle_adaptor.dir/build.make:76: CMakeFiles/vehicle_adaptor.dir/src/vehicle_adaptor/vehicle_adaptor.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:189: CMakeFiles/vehicle_adaptor.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2
---

Signed-off-by: kosuke55 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:vehicle Vehicle-specific implementations, drivers, packages. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) type:documentation Creating or refining documentation. (auto-assigned)
Projects
Status: To Triage
Development

Successfully merging this pull request may close these issues.

2 participants