Skip to content
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

Enable unit tests #453

Closed
wants to merge 1 commit into from
Closed

Conversation

aacostadiaz
Copy link
Collaborator

This pull request enabled the unit tests that were disabled due to a bug in dpcpp (intel/llvm#7075).

@aacostadiaz aacostadiaz changed the base branch from master to aacosta/dpcpp_version August 2, 2023 09:11
@aacostadiaz aacostadiaz added the stacked PR is stacked on top of another label Aug 2, 2023
@aacostadiaz
Copy link
Collaborator Author

Closing this pull request because the daily version still has the bug. It works fine with the official release version (tested with 2023.2.0 and 2023.3.0). The assert that is failing in the daily version was probably removed in the release version.

@aacostadiaz aacostadiaz closed this Aug 2, 2023
@muhammad-tanvir-1211 muhammad-tanvir-1211 deleted the aacosta/enable_tests branch October 23, 2023 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stacked PR is stacked on top of another
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant