Skip to content

Commit

Permalink
fix(boojum-cuda): "un-swap" coarse and fine count for powers_data_g_i (
Browse files Browse the repository at this point in the history
…#27)

# What ❔

Fixes bug in context creation where `coarse_log_count` and
`fine_log_count` were mistakenly swapped.

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [x] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [x] Tests for the changes have been added / updated.
- [x] Documentation comments have been added / updated.
- [x] Code has been formatted via `zk fmt` and `zk lint`.
  • Loading branch information
mcarilli authored Sep 10, 2024
1 parent a099924 commit 95b29da
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions crates/boojum-cuda/src/context.rs
Original file line number Diff line number Diff line change
Expand Up @@ -132,9 +132,9 @@ unsafe fn copy_to_symbols(
&powers_data_g_i,
&PowersData::new(
powers_of_g_i_fine,
coarse_log_count,
powers_of_g_i_coarse,
fine_log_count,
powers_of_g_i_coarse,
coarse_log_count,
),
)?;
copy_to_symbol(&inv_sizes, &inv_sizes_host)?;
Expand Down

0 comments on commit 95b29da

Please sign in to comment.