Skip to content

馃摑 Add docstrings to feature/pnp-solver-develop - #13

Closed
coderabbitai[bot] wants to merge 1 commit into
feature/pnp-solver-developfrom
coderabbitai/docstrings/f959bcb
Closed

馃摑 Add docstrings to feature/pnp-solver-develop#13
coderabbitai[bot] wants to merge 1 commit into
feature/pnp-solver-developfrom
coderabbitai/docstrings/f959bcb

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Docstrings generation was requested by @heyeuu.

The following files were modified:

  • src/component.cpp
  • src/kernel/pose_estimator.cpp
  • src/kernel/visualization.cpp
  • src/kernel/visualization.hpp
  • src/module/capturer/local_video.cpp
  • src/module/debug/visualization/armor_visualizer.cpp
  • src/module/identifier/model.cpp
  • src/runtime.cpp
  • src/utility/math/conversion.hpp
  • src/utility/math/linear.hpp
  • src/utility/math/solve_pnp/pnp_solution.cpp
  • src/utility/math/solve_pnp/pnp_solution.hpp
  • src/utility/math/solve_pnp/solve_pnp.hpp
  • src/utility/model/armor_detection.hpp
  • src/utility/rclcpp/node.details.hpp
  • src/utility/rclcpp/visual/armor.cpp
  • src/utility/rclcpp/visual/armor.hpp
  • src/utility/robot/armor.hpp
  • src/utility/robot/id.hpp
  • src/utility/tf/static_tf.hpp
  • test/assets_manager.hpp
  • test/download_assets.sh
  • test/solve_pnp.cpp
  • test/transform_communication.cpp
  • tool/visualization.cpp
These files were kept as they were
  • test/model_infer.cpp
  • test/static_tf.cpp
These file types are not supported
  • .github/workflows/gtest.yml
  • README.md
  • config/config.yaml
  • package.xml
  • test/CMakeLists.txt
  • test/asset.yml
  • tool/CMakeLists.txt
鈩癸笍 Note

CodeRabbit cannot perform edits on its own pull requests yet.

Docstrings generation was requested by @heyeuu.

* #12 (comment)

The following files were modified:

* `src/component.cpp`
* `src/kernel/pose_estimator.cpp`
* `src/kernel/visualization.cpp`
* `src/kernel/visualization.hpp`
* `src/module/capturer/local_video.cpp`
* `src/module/debug/visualization/armor_visualizer.cpp`
* `src/module/identifier/model.cpp`
* `src/runtime.cpp`
* `src/utility/math/conversion.hpp`
* `src/utility/math/linear.hpp`
* `src/utility/math/solve_pnp/pnp_solution.cpp`
* `src/utility/math/solve_pnp/pnp_solution.hpp`
* `src/utility/math/solve_pnp/solve_pnp.hpp`
* `src/utility/model/armor_detection.hpp`
* `src/utility/rclcpp/node.details.hpp`
* `src/utility/rclcpp/visual/armor.cpp`
* `src/utility/rclcpp/visual/armor.hpp`
* `src/utility/robot/armor.hpp`
* `src/utility/robot/id.hpp`
* `src/utility/tf/static_tf.hpp`
* `test/assets_manager.hpp`
* `test/download_assets.sh`
* `test/solve_pnp.cpp`
* `test/transform_communication.cpp`
* `tool/visualization.cpp`
@coderabbitai

coderabbitai Bot commented Dec 16, 2025

Copy link
Copy Markdown
Contributor Author

Important

Review skipped

CodeRabbit bot authored PR detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@heyeuu heyeuu closed this Dec 16, 2025
@github-project-automation github-project-automation Bot moved this from Todo to Done in RMCS Auto Aim V2 Dec 16, 2025
@heyeuu
heyeuu deleted the coderabbitai/docstrings/f959bcb branch December 17, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant