Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow deriveColumnDescriptions to use first n rows for deriving #636

Open
puehringer opened this issue Aug 30, 2023 · 0 comments
Open

Allow deriveColumnDescriptions to use first n rows for deriving #636

puehringer opened this issue Aug 30, 2023 · 0 comments
Assignees
Labels
type: feature New feature or request

Comments

@puehringer
Copy link
Contributor

It would be great if the deriveColumns functions of the builder would not just use the first = data[0] row, but n rows. This could be passed as IDeriveOptions for example, with n = 1 staying the default. I think we can keep it very simple, i.e. by letting deriveType return a flag if the returned column is the "default"/not inferred one, and we keep doing this until we reach n iterations or find a non-default column.

User story

Generally, this is needed when you have columns with missing values, i.e. [null, 'https://...', ...]. This will cause the default to be triggered (i.e. returning a string column).

@puehringer puehringer added the type: feature New feature or request label Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants