Use setBackend instead of ConvertToCpu workaround
Change the convertion to backend 'cpu' from a custom function converToCpu
in the database to the generic setBackend()
.
Content :
-
Remove the call to convertToCpu in database, its definition & tests.
However it raises a problem on the layout of the data between different backends.
In the current code the resulting tensor with backend 'cpu' have layout [N,H,W,C] instead of [N,C,H,W].
-
Use format HWC for OpenCV tensor and change format when converting to CPU backend.
Merge request reports
Activity
changed milestone to %aidge_backend_opencv - v0.1.0
added LanguageC++ label
requested review from @olivierbichler and @pineapple
assigned to @thibaultallenet
added StatusWIP label
removed milestone %aidge_backend_opencv - v0.1.0
@thibaultallenet Looks like this MR is outdated, yet the
convertCpu
is still there. Is there any chance to quickly finalize this MR now that there is data layout support in core?removed StatusWork in Progress label
added StatusReview Ready label
enabled an automatic merge when all merge checks for f4e82ab8 pass
mentioned in issue #1 (closed)
mentioned in commit 3da8f80b