Skip to content

Fix problem in bli_obj_imag_part.#861

Merged
devinamatthews merged 1 commit intomasterfrom
imag_part
Mar 2, 2025
Merged

Fix problem in bli_obj_imag_part.#861
devinamatthews merged 1 commit intomasterfrom
imag_part

Conversation

@devinamatthews
Copy link
Copy Markdown
Member

@devinamatthews devinamatthews commented Mar 2, 2025

Details:

  • When adjusting the buffer to point to the first imaginary element, the function bli_obj_buffer_at_off was used which includes any currently set offsets, but then bli_obj_set_buffer was used which is the buffer before applying offsets.
  • Now a matching bli_obj_buffer is used to avoid any offsets.

Details:
- When adjusting the buffer to point to the first imaginary element, the function `bli_obj_buffer_at_off` was used which includes and currently set offsets, but then `bli_obj_set_buffer` was used which is the offset *before* applying offsets.
- Now a matching `bli_obj_buffer` is used to avoid any offsets.
@devinamatthews devinamatthews merged commit 97084c7 into master Mar 2, 2025
@devinamatthews devinamatthews deleted the imag_part branch March 2, 2025 14:50
devinamatthews added a commit that referenced this pull request Jun 7, 2025
Details:
- When adjusting the buffer to point to the first imaginary element, the function `bli_obj_buffer_at_off` was used which includes and currently set offsets, but then `bli_obj_set_buffer` was used which is the offset *before* applying offsets.
- Now a matching `bli_obj_buffer` call is used to avoid any offsets.

(cherry picked from commit 97084c7)
devinamatthews added a commit that referenced this pull request Jun 7, 2025
Details:
- When adjusting the buffer to point to the first imaginary element, the function `bli_obj_buffer_at_off` was used which includes and currently set offsets, but then `bli_obj_set_buffer` was used which is the offset *before* applying offsets.
- Now a matching `bli_obj_buffer` call is used to avoid any offsets.

(cherry picked from commit 97084c7)
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