Skip to content

feat(c3) : Add ability to evaluate cost of C3 plan#44

Draft
ebianchi wants to merge 4 commits intomainfrom
bibit/cost-evaluation
Draft

feat(c3) : Add ability to evaluate cost of C3 plan#44
ebianchi wants to merge 4 commits intomainfrom
bibit/cost-evaluation

Conversation

@ebianchi
Copy link
Contributor

@ebianchi ebianchi commented Feb 23, 2026

Addresses #31


This change is Reviewable

@github-actions
Copy link

github-actions bot commented Feb 23, 2026

LCOV of commit df22ca5 during C3 Coverage #92

Summary coverage rate:
  lines......: 92.8% (1644 of 1772 lines)
  functions..: 78.0% (135 of 173 functions)
  branches...: 52.6% (1280 of 2434 branches)

Files changed coverage rate:
                                            |Lines       |Functions  |Branches    
  Filename                                  |Rate     Num|Rate    Num|Rate     Num
  ================================================================================
  core/c3.cc                                | 8.0%    276|2171%    21|    -      0
  core/c3.h                                 |18.2%     22|1400%     3|    -      0
  core/lcs.cc                               |13.7%     51|1167%     6|    -      0
  core/lcs.h                                |17.6%     17|2667%     3|    -      0

@Meow404
Copy link
Collaborator

Meow404 commented Feb 25, 2026

The overall structure really looks good to me 🔥 . I haven't taken a deep dive, but having some nested namespaces for trajectory evaluation would make sure they are well scoped. Letting the functions be declared static as part of a class is also an option but not required.

@Meow404 Meow404 linked an issue Feb 25, 2026 that may be closed by this pull request
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.

Refine Cost Calculation in C3 Library

2 participants