feat: add content types

[ianna]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 96.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 75.29%. Comparing base (d11e61d) to head (6769ac1).
Report is 7 commits behind head on main.

Files	Patch %	Lines
src/all_implementations.jl	95.45%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
+ Coverage   74.06%   75.29%   +1.22%     
==========================================
  Files           4        4              
  Lines        1666     1623      -43     
==========================================
- Hits         1234     1222      -12     
+ Misses        432      401      -31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Moelf]

sorry I should have left a comment earlier. I wonder if this is the best way to go about it. I suppose this is a way to fix things, I don't know the performance implication.

An alternative way is to fix:

AwkwardArray.jl/src/all_implementations.jl

Line 101 in d11e61d

abstract type Content{BEHAVIOR} <: AbstractVector{Any} end

by adding one more parametric type into it, so that it can be properly <:AbstractArray{T, 1}?

[Moelf]

yeah looking into it right now, I opened #66

[jpivarski]

If I'm understanding the eltype correctly—that it's the type of an element in the array (the return type of getindex)—then some of these have been set incorrectly. See below.

[jpivarski]

Does a Record (a scalar object, not an array) even have an eltype?

[Moelf]

I think it depends on if you model the Record as an indexable thing. For me at least, I think event-based table semantically as a row table, so the eltype is whatever the type an "event" would give you

[ianna]

I'm not sure if it makes sense to separate FIELDS and CONTENTS, but perhaps there is a usecase for it?

julia> typeof(layout[3])
AwkwardArray.Record{(:a, :b), Tuple{AwkwardArray.PrimitiveArray{Int64, Vector{Int64}, :default}, AwkwardArray.ListOffsetArray{Vector{Int64}, AwkwardArray.PrimitiveArray{Float64, Vector{Float64}, :default}, :default}}, :default}

julia> r = layout[3]
{a: 3, b: [4.4, 5.5]}

julia> eltype(r)
Tuple{AwkwardArray.PrimitiveArray{Int64, Vector{Int64}, :default}, AwkwardArray.ListOffsetArray{Vector{Int64}, AwkwardArray.PrimitiveArray{Float64, Vector{Float64}, :default}, :default}}

[ianna]

I've added some more tests unrelated to this PR to make the coverage requirement pass.

[ianna]

@jpivarski - I changed that to allow a copy beneath. The need for the behavior::Symbol showed in a new test.