Closed tpapp closed 4 years ago
I am happy to write this, just wanted to discuss first as it is a breaking API change, and would require that we bump to 2.0. I made #233 so that the users who want to wait before updating get a working version in the meantime.
Why does it have to be a breaking change?
Many packages now implement this abstract interface, and it could serve as the fallback for
TableData
, simplifying the code a lot. Specifically,TableData(table; scanlines = 0, rowsep = ROWSEP)
AbstractMatrix
, a vector ofNamedTuple
s, aNamedTuple
of vectors in place of the methods currently used by the package,We could just depend on Tables.jl as it is rather lightweight, alleviating #227.