Skip to content

Conversation

@kozlov721
Copy link
Collaborator

Purpose

Makes it easier to append to list using CLI overrides for a config.

Specification

The following syntax is now supported in config overrides:

cofig.list_field.+.new_value

which will append new_value to the list_field.

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

None / not applicable

@kozlov721 kozlov721 requested a review from a team as a code owner August 1, 2025 23:23
@kozlov721 kozlov721 requested review from conorsim, klemen1999 and tersekmatija and removed request for a team August 1, 2025 23:23
@github-actions github-actions bot added enhancement New feature or request utils Changes affecting luxonis_ml.utils subpackage labels Aug 1, 2025
@codecov
Copy link

codecov bot commented Aug 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.30%. Comparing base (64d29f7) to head (0fa0fbc).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #359      +/-   ##
==========================================
+ Coverage   95.26%   95.30%   +0.04%     
==========================================
  Files         103      103              
  Lines        6333     6393      +60     
==========================================
+ Hits         6033     6093      +60     
  Misses        300      300              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kozlov721 kozlov721 merged commit 8ea28dd into main Aug 4, 2025
16 checks passed
@kozlov721 kozlov721 deleted the feat/config-list-append branch August 4, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request utils Changes affecting luxonis_ml.utils subpackage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants