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

Block global kron cpu #687

Merged
merged 27 commits into from
Jun 25, 2024

Conversation

mkstoyanov
Copy link
Collaborator

Block-global kronmult will be used for interpolation.

@mkstoyanov mkstoyanov force-pushed the block_global_kron_cpu branch 2 times, most recently from 627b8e5 to b06e2b4 Compare June 21, 2024 11:03
@mkstoyanov mkstoyanov marked this pull request as ready for review June 21, 2024 14:47
@mkstoyanov
Copy link
Collaborator Author

The block-global kronmult is faster than the original one so we will replace the old version but only after we get the GPU kernels. Right now, we need the extra CMake option -DKRON_MODE_GLOBAL_BLOCK that only work on the CPU side but will be removed in the future.

@mkstoyanov mkstoyanov force-pushed the block_global_kron_cpu branch from f347875 to df836f0 Compare June 25, 2024 17:17
quantumsteve
quantumsteve previously approved these changes Jun 25, 2024
@mkstoyanov
Copy link
Collaborator Author

@quantumsteve Sorry but I put some pretty lines here and there, should be good for your re-review now. Nothing major has changed.

@mkstoyanov mkstoyanov merged commit 5c37468 into project-asgard:develop Jun 25, 2024
11 checks passed
@mkstoyanov mkstoyanov deleted the block_global_kron_cpu branch June 25, 2024 18:54
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.

2 participants