Skip to content

Feat/triangulation_weight_id#15

Merged
CurryMonsters merged 14 commits intomainfrom
feat/triangulation_weight_id
Mar 29, 2026
Merged

Feat/triangulation_weight_id#15
CurryMonsters merged 14 commits intomainfrom
feat/triangulation_weight_id

Conversation

@CurryMonsters
Copy link
Copy Markdown
Member

added possibility for having multiple target ids, added weights for calculations, fixed tf issue.

@CurryMonsters CurryMonsters requested a review from jorgenfj March 29, 2026 09:23
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 29, 2026

Codecov Report

❌ Patch coverage is 0% with 44 lines in your changes missing coverage. Please review.
✅ Project coverage is 12.24%. Comparing base (f860da4) to head (c8956f9).
⚠️ Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
...localization/src/ros/bearing_localization_node.cpp 0.00% 44 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   11.48%   12.24%   +0.76%     
==========================================
  Files          28       25       -3     
  Lines        1271     1192      -79     
  Branches      297      293       -4     
==========================================
  Hits          146      146              
+ Misses       1039      960      -79     
  Partials       86       86              
Flag Coverage Δ
unittests 12.24% <0.00%> (+0.76%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...g_localization/lib/bearing_localization_config.hpp 0.00% <ø> (ø)
...localization/src/ros/bearing_localization_node.cpp 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@jorgenfj jorgenfj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm.
Just remove the pipeline fsm that was accidentally included

@CurryMonsters CurryMonsters merged commit 760dbf5 into main Mar 29, 2026
5 checks passed
@CurryMonsters CurryMonsters deleted the feat/triangulation_weight_id branch March 29, 2026 18:38
@CurryMonsters CurryMonsters restored the feat/triangulation_weight_id branch March 29, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants