Closed roedoejet closed 2 months ago
Review changes with SemanticDiff.
Analyzed 3 of 3 files.
Overall, the semantic diff is 29% smaller than the GitHub diff.
Filename | Status | |
---|---|---|
:heavy_check_mark: | everyvoice/cli.py | 47.39% smaller |
:heavy_check_mark: | everyvoice/tests/test_preprocessing.py | Analyzed |
:heavy_check_mark: | everyvoice/preprocessor/preprocessor.py | 34.87% smaller |
Attention: Patch coverage is 60.00000%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 74.36%. Comparing base (
b40375a
) to head (0ac0b6c
).
Files | Patch % | Lines |
---|---|---|
everyvoice/preprocessor/preprocessor.py | 61.53% | 1 Missing and 4 partials :warning: |
everyvoice/cli.py | 50.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
CLI load time: 0:00.23
Pull Request HEAD: 0ac0b6c5c9192bebf722f6ac3d3223f0da673636
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
PR Goal?
Incorporate TorchSquim metrics into EveryVoice data checking
Fixes?
Feedback sought?
Sanity check. Review functionality
Priority?
low
Tests added?
None, we should add some tests to check that the data checker is working basically
How to test?
everyvoice check-data <path_to_config> --heavy-objective-evaluation
Confidence?
medium
Version change?
No
Related PRs?