-
Notifications
You must be signed in to change notification settings - Fork 55
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
DOC: Add Codecov badge to README #343
Conversation
@ant0nsc do you know why Codecov might be reporting a large difference in coverage for this? Also, why is coverage uploaded twice in the build workflow? I missed that in the PR. hi-ml/.github/workflows/build-test-pr.yml Lines 207 to 211 in 7ddb45d
hi-ml/.github/workflows/build-test-pr.yml Lines 262 to 266 in 7ddb45d
|
Apparently, the difference shown here might have been "a timing issue". |
@fepegar , Uploading is happening twice because first we do it for hi-ml and hi-ml-azure (matrix), and then in line 262 for the histo folder. However... For this run, there is no sign of results from hi-ml-azure? (screenshot from https://app.codecov.io/gh/microsoft/hi-ml/compare/343/tree) |
Thanks for clarifying. No idea why that happened! It looks ok for more recent commits. |
It looks like this: