woodwork icon indicating copy to clipboard operation
woodwork copied to clipboard

Separate `include` parameter in `_filter_cols` into tags, ltypes, and column names

Open tamargrey opened this issue 4 years ago • 0 comments

WoodworkTableAccessor._filter_cols is used by select and describe, and it takes in a single include parameter that tries to determine what is a semantic tag vs logical type vs column name. This has left us with some pretty complicated code that doesn't really do a perfect job.

We should separate the include parameter into logical_types, semantic_tags, and column_names to simplify things.

tamargrey avatar Feb 12 '21 20:02 tamargrey