Remove access to underlying contiguous TL buffer in Flip op

[klecki]

Description

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Refactoring (Redesign of existing code that doesn't affect functionality)
• Other (e.g. Documentation, Tests, Configuration)

What happened in this PR

Remove only the mutable_data and data usage
in GPU variant.

Tested with enforced shapes.
This op still uses per-sample approach in both
CPU and GPU variants, this is left unchanged.

Additional information

• Affected modules and functionalities:
  Flip

• Key points relevant for the review:

Checklist

Tests

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

Documentation

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

DALI team only

Requirements

• Implements new requirements
• Affects existing requirements
• N/A

REQ IDs: N/A

JIRA TASK: N/A

[klecki]

!build

[dali-automaton]

CI MESSAGE: [2833186]: BUILD STARTED

[dali-automaton]

CI MESSAGE: [2833186]: BUILD PASSED

[szalpal]

Is it justified to have multiple statements per line? Generally we'd rather avoid such...

I'd be in favour of extracting these to a separate function:

template<typename DType>
void f(...) {
      auto in_shape = TransformShapes(input.shape(), input.GetLayout());
      auto in_view = reshape<flip_ndim>(view<const DType>(input), in_shape);
      kernels::KernelContext ctx;
      ctx.gpu.stream = stream;
      kernels::FlipGPU<DType> kernel;
      auto reqs = kernel.Setup(ctx, in_view);
      auto out_view =
          reshape<flip_ndim>(view<DType>(output), reqs.output_shapes[0].to_static<flip_ndim>());
      kernel.Run(ctx, out_view, in_view, depthwise, vertical, horizontal);)
}

[klecki]

I have no idea why it happened here, probably due to the old DALI_TYPE_SWITCH the formatter got confused.

I will fix it, I don't think we need additional function though.

[klecki]

!build

[dali-automaton]

CI MESSAGE: [2848854]: BUILD STARTED

[dali-automaton]

CI MESSAGE: [2848854]: BUILD PASSED