computablee / DotMP

A collection of powerful abstractions for parallel programming in .NET with an OpenMP-like API.
https://computablee.github.io/DotMP/
GNU Lesser General Public License v2.1
29 stars 7 forks source link

Add more tests for reductions and bug fix #82

Closed computablee closed 1 year ago

computablee commented 1 year ago

This PR adds more tests for DotMP.Parallel.ForReduction, and implements a bug fix that the new tests caught.

codecov[bot] commented 1 year ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (63ecd8e) 85.09% compared to head (62b7900) 89.31%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #82 +/- ## ========================================== + Coverage 85.09% 89.31% +4.22% ========================================== Files 11 11 Lines 805 805 Branches 85 86 +1 ========================================== + Hits 685 719 +34 + Misses 96 64 -32 + Partials 24 22 -2 ``` | [Files](https://app.codecov.io/gh/computablee/DotMP/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Phillip+Allen+Lane) | Coverage Δ | | |---|---|---| | [DotMP/Init.cs](https://app.codecov.io/gh/computablee/DotMP/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Phillip+Allen+Lane#diff-RG90TVAvSW5pdC5jcw==) | `100.00% <100.00%> (+17.89%)` | :arrow_up: | | [DotMP/Parallel.cs](https://app.codecov.io/gh/computablee/DotMP/pull/82?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Phillip+Allen+Lane#diff-RG90TVAvUGFyYWxsZWwuY3M=) | `79.78% <ø> (+4.64%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.