Skip to content

Add the coarse library #931

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

Merged
merged 2 commits into from
Dec 2, 2024
Merged

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Nov 25, 2024

Description

Add the coarse library that will replace the coarse provider.

Example of usage: #932

Code coverage of the coarse library is 92.6%.
image

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau requested a review from a team as a code owner November 25, 2024 11:24
@ldorau ldorau requested a review from bratpiorka November 25, 2024 11:27
@ldorau ldorau force-pushed the Add_the_coarse_library branch from 0747074 to bf3610b Compare November 25, 2024 12:24
@ldorau
Copy link
Contributor Author

ldorau commented Nov 25, 2024

Why do we need it as a dedicated library? We need an arch discussion if we would like to introduce one more library.

It is just a static inner-helper library. There already was an arch discussion about this.

@ldorau
Copy link
Contributor Author

ldorau commented Nov 25, 2024

Example of usage: #932

@ldorau ldorau requested a review from vinser52 November 25, 2024 13:13
@ldorau
Copy link
Contributor Author

ldorau commented Nov 25, 2024

Code coverage of the coarse library is 92.6%.
image

@ldorau
Copy link
Contributor Author

ldorau commented Nov 26, 2024

@bratpiorka @lplewa please review

@bratpiorka
Copy link
Contributor

overall LGTM. nit: please set year in newly added files to 2024

@ldorau ldorau force-pushed the Add_the_coarse_library branch from 0897b48 to 814185b Compare November 28, 2024 11:38
@ldorau
Copy link
Contributor Author

ldorau commented Nov 28, 2024

overall LGTM. nit: please set year in newly added files to 2024

Done

@ldorau ldorau force-pushed the Add_the_coarse_library branch from 814185b to c2fc3db Compare November 29, 2024 07:44
@ldorau
Copy link
Contributor Author

ldorau commented Nov 29, 2024

Rebased

@ldorau ldorau force-pushed the Add_the_coarse_library branch from c2fc3db to 7dab2ec Compare December 2, 2024 07:22
Add the coarse library that will replace the coarse provider.

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Add_the_coarse_library branch from 7dab2ec to 958b690 Compare December 2, 2024 07:38
@ldorau ldorau requested a review from vinser52 December 2, 2024 07:39
@bratpiorka bratpiorka merged commit 6987891 into oneapi-src:main Dec 2, 2024
77 checks passed
@ldorau ldorau deleted the Add_the_coarse_library branch December 2, 2024 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants