Fix Shift bug where there was an empty dimension in Array by umar456 · Pull Request #3488 · arrayfire/arrayfire · GitHub
Skip to content

Fix Shift bug where there was an empty dimension in Array#3488

Merged
umar456 merged 2 commits intoarrayfire:masterfrom
umar456:shift_fix
Aug 19, 2023
Merged

Fix Shift bug where there was an empty dimension in Array#3488
umar456 merged 2 commits intoarrayfire:masterfrom
umar456:shift_fix

Conversation

@umar456
Copy link
Copy Markdown
Member

@umar456 umar456 commented Aug 19, 2023

Fix bug in shift where there is an empty dimension in array(i.e. 10, 1, 10). This caused an error because of the new compaction of the dimensions in the JIT code generation.

Description

  • Fixes bug in JIT kernel generation when there is a singleton dimension in the Array
  • This also addresses an issue in reorder where the reorder function wasn't evaluated.

Fixes: #3475

Changes to Users

N/A

Checklist

  • Rebased on latest master
  • Code compiles
  • Tests pass
  • Functions added to unified API
  • Functions documented

The reorder funciton was copying the Array object internally and
then the other operations were performed on the copy. This causes
the eval to be performed on the copied array instead of the input
array.
@umar456 umar456 merged commit 23ee065 into arrayfire:master Aug 19, 2023
@umar456 umar456 deleted the shift_fix branch August 19, 2023 20:44
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.

[BUG] Shift returns incorrect results with arrays with skipped dimensions

2 participants