Skip to content

Conversation

@allnes
Copy link
Member

@allnes allnes commented Jan 19, 2025

No description provided.

@allnes allnes force-pushed the an/simplify_examples branch from 80ccfa4 to a61d117 Compare January 19, 2025 17:15
@allnes allnes self-assigned this Jan 19, 2025
@allnes allnes force-pushed the an/simplify_examples branch 3 times, most recently from 3e6fa20 to 4e1b999 Compare January 19, 2025 17:36
@allnes allnes force-pushed the an/simplify_examples branch 4 times, most recently from f7a63ab to d0fe9b0 Compare January 19, 2025 20:18
@allnes allnes force-pushed the an/simplify_examples branch from d0fe9b0 to 442ac45 Compare January 19, 2025 20:19
@allnes
Copy link
Member Author

allnes commented Jan 19, 2025

@aobolensk ready for review

@codecov
Copy link

codecov bot commented Jan 19, 2025

Codecov Report

Attention: Patch coverage is 92.46575% with 11 lines in your changes missing coverage. Please review.

Project coverage is 93.94%. Comparing base (8b7417b) to head (a8a7c4e).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
tasks/all/example/src/ops_all.cpp 83.78% 5 Missing and 1 partial ⚠️
tasks/mpi/example/src/ops_mpi.cpp 81.81% 4 Missing ⚠️
modules/core/util/util.hpp 87.50% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #236      +/-   ##
==========================================
+ Coverage   93.32%   93.94%   +0.62%     
==========================================
  Files          34       26       -8     
  Lines         794      479     -315     
  Branches      222      143      -79     
==========================================
- Hits          741      450     -291     
- Misses          9       14       +5     
+ Partials       44       15      -29     

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

@allnes allnes force-pushed the an/simplify_examples branch from 7854864 to a8a7c4e Compare January 19, 2025 22:20
@aobolensk aobolensk added this pull request to the merge queue Jan 20, 2025
Merged via the queue into learning-process:master with commit ca057a4 Jan 20, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simplify examples

2 participants