Fix Windows Compilation Issues in joint_trajectory_controller and pid_controller

[SENAI-GilmarCorreia]

Description:

This pull request addresses compilation issues encountered in the joint_trajectory_controller and pid_controller when building on Windows. The changes ensure compatibility with the Windows environment and improve overall stability.

Changes Made:

• joint_trajectory_controller:

  • Updated includes to resolve path issues specific to Windows.
  • Modified conditional compilation flags to ensure compatibility with Windows builds.

• pid_controller:

  • Fixed issues related to the use of specific libraries that were not resolving correctly on Windows.
  • Added compile definitions for Windows to minimize namespace collisions and define math constants:

    if(WIN32)
      add_compile_definitions(
        # For math constants
        _USE_MATH_DEFINES
        # Minimize Windows namespace collision
        NOMINMAX
        WIN32_LEAN_AND_MEAN
      )
    endif()

  • Ensured that necessary headers are included for successful compilation.

Testing:

• Local tests have been run successfully, confirming that all modifications do not break existing functionality.
• New tests have been added to cover the changes made in both controllers, ensuring they work as intended on Windows.

Please let me know if you have any questions or need further modifications. I appreciate your feedback!

[mergify]

@SENAI-GilmarCorreia, all pull requests must be targeted towards the master development branch.
Once merged into master, it is possible to backport to humble, but it must be in master
to have these changes reflected into new distributions.

[christophfroehlich]

closing in favor of #1330 -> we will backport that then