Open hasan7n opened 2 months ago
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅
Attention: Patch coverage is 35.71429%
with 9 lines
in your changes missing coverage. Please review.
Project coverage is 93.52%. Comparing base (
c6d178f
) to head (ac2c98b
).
Files with missing lines | Patch % | Lines |
---|---|---|
GANDLF/cli/generate_metrics.py | 35.71% | 9 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Fixes #ISSUE_NUMBER
Proposed Changes
Checklist
CONTRIBUTING
guide has been followed.typing
is used to provide type hints, including and not limited to usingOptional
if a variable has a pre-defined value).pip install
step is needed for PR to be functional), please ensure it is reflected in all the files that control the CI, namely: python-test.yml, and all docker files [1,2,3].logging
library is being used and noprint
statements are left.