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

Update benchmarks #38

Merged
merged 1 commit into from
Oct 4, 2024
Merged

Update benchmarks #38

merged 1 commit into from
Oct 4, 2024

Conversation

charleskawczynski
Copy link
Member

@charleskawczynski charleskawczynski commented Oct 3, 2024

This PR

  • Removes one of the deep copies, so that we have enough heap memory on the device
  • Reduces the complexity of the kernels
  • Adds degrees of freedom to the arrays
  • Corrects some of the theoretical reads/writes for the kernels

@charleskawczynski charleskawczynski force-pushed the ck/update_bm branch 2 times, most recently from 10fb836 to 62913a2 Compare October 3, 2024 19:53
@charleskawczynski charleskawczynski changed the title Update benchmarks problem size Update benchmarks Oct 3, 2024
@charleskawczynski
Copy link
Member Author

Not sure why MacOS is breaking, looks unrelated to the changes.

@charleskawczynski charleskawczynski merged commit 86c3392 into main Oct 4, 2024
8 of 10 checks passed
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.

1 participant