Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

Conversation

@1uc
Copy link
Collaborator

@1uc 1uc commented May 6, 2024

No description provided.

@1uc 1uc force-pushed the 1uc/remove-broadcasting branch from 2b1158b to 66468f0 Compare May 8, 2024 07:37
@1uc 1uc force-pushed the 1uc/remove-broadcasting branch from 66468f0 to 3489d48 Compare May 9, 2024 12:08
@codecov-commenter
Copy link

codecov-commenter commented May 9, 2024

Codecov Report

Attention: Patch coverage is 95.55556% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 86.48%. Comparing base (abf4c69) to head (5bfa033).

❗ Current head 5bfa033 differs from pull request most recent head a7fa92f. Consider uploading reports for the commit a7fa92f to get more accurate results

Files Patch % Lines
include/highfive/bits/assert_compatible_spaces.hpp 62.50% 3 Missing ⚠️
src/examples/broadcasting_arrays.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #992      +/-   ##
==========================================
+ Coverage   86.29%   86.48%   +0.19%     
==========================================
  Files          96       95       -1     
  Lines        5998     5951      -47     
==========================================
- Hits         5176     5147      -29     
+ Misses        822      804      -18     

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

@1uc 1uc force-pushed the 1uc/remove-broadcasting branch from 600d68f to 5bfa033 Compare May 10, 2024 13:29
@1uc 1uc force-pushed the 1uc/remove-broadcasting branch from 5bfa033 to a7fa92f Compare May 13, 2024 14:31
@1uc 1uc marked this pull request as ready for review May 13, 2024 15:03
@ferdonline
Copy link
Contributor

LGTM

@1uc 1uc merged commit 5bd727d into master May 14, 2024
@1uc 1uc deleted the 1uc/remove-broadcasting branch May 14, 2024 05:51
@1uc 1uc mentioned this pull request May 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants