DALI icon indicating copy to clipboard operation
DALI copied to clipboard

Use AccessOrder in TypeInfo::Copy. Add synchronization when specifying host order for x2D and D2x copy.

Open mzient opened this issue 7 months ago • 3 comments

Category:

Bug fix (non-breaking change which fixes an issue) Refactoring (Redesign of existing code that doesn't affect functionality)

Description:

Additional information:

Before this change specifying AccessOrder::host in x2D pr D2x copy resulted in missing synchronization. It should be either forbidden or properly synchronized.

Affected modules and functionalities:

Tensor & TensorList copying.

Key points relevant for the review:

Tests:

  • [X] Existing tests apply
  • [ ] New tests added
    • [ ] Python tests
    • [ ] GTests
    • [ ] Benchmark
    • [ ] Other
  • [ ] N/A

Checklist

Documentation

  • [ ] Existing documentation applies
  • [ ] Documentation updated
    • [ ] Docstring
    • [ ] Doxygen
    • [ ] RST
    • [ ] Jupyter
    • [ ] Other
  • [X] N/A

DALI team only

Requirements

  • [ ] Implements new requirements
  • [ ] Affects existing requirements
  • [X] N/A

REQ IDs: N/A

JIRA TASK: N/A

mzient avatar Jul 10 '24 16:07 mzient