Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Forbid Extra Fields #13

Merged
merged 2 commits into from
Feb 22, 2024
Merged

Forbid Extra Fields #13

merged 2 commits into from
Feb 22, 2024

Conversation

kozlov721
Copy link
Collaborator

Updated the config so extra keys are now forbidden.

@kozlov721 kozlov721 added the fix Fixing a bug label Feb 22, 2024
Copy link

github-actions bot commented Feb 22, 2024

Test Results

  6 files    6 suites   1h 6m 21s ⏱️
 32 tests  32 ✅ 0 💤 0 ❌
192 runs  192 ✅ 0 💤 0 ❌

Results for commit c8e0d16.

♻️ This comment has been updated with latest results.

Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
4435 3531 80% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
luxonis_train/utils/config.py 95% 🟢
TOTAL 95% 🟢

updated for commit: c8e0d16 by action🐍

@kozlov721 kozlov721 merged commit 2c62a08 into dev Feb 22, 2024
10 checks passed
@kozlov721 kozlov721 deleted the fix/config-extra-fields branch February 22, 2024 07:57
kozlov721 added a commit that referenced this pull request Oct 9, 2024
* forbid extra fields in config

* fixed configs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant