JuliaSIMD / JuliaSIMD/LoopVectorization.jl

Support for RGB colors

Open
#67 9 comments 0 reactions 0 assignees View on GitHub
Dominant language
Julia
Stars
789
Forks
73
PR merge metrics
No merged PRs in 30d

Description

Aside from #66, I think one of the last obstacles to using this in ImageFiltering is the handling of color array elements, for example `RGB(0.8f0, 0.3f0, 0.5f0)` (which is just a struct of 3 `Float32`s). Through TiledIteration it's possible to convert the data into a suitable format, for example `RGB{N0f8}` can be converted to `RGB{Float32}` because [`TileBuffer` takes an element type as an argument](https://github.com/JuliaArrays/TiledIteration.jl#allocating-and-managing-temporary-storage). (I note that's really old syntax, with modern Julia I should change it to `TileBuffer{T}(undef, tileaxs)`.) Consequently it will be converted before it is used; because convolution/correlation uses each element multiple times, this conversion is well worth doing if it gets us better performance and allows us to use this package. (Another benefit of copying is that we can guarantee stridedness.)

So the question is, what kind of format should I convert it to? Should we define explict support for RGB, or would it be better to convert to an `SVector{3}` or similar? I kind of favor the latter since it would presumably be more generic.

I suspect this will alter the loop unrolling/SIMD packing, once I understand what representation you'd like here.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.