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

TSV import always trims white space, ignoring parse setting #6604

Closed
jquartel opened this issue May 14, 2024 · 0 comments · Fixed by #6609
Closed

TSV import always trims white space, ignoring parse setting #6604

jquartel opened this issue May 14, 2024 · 0 comments · Fixed by #6609
Assignees
Labels
CSV/TSV About the CSV/TSV import or export import About importers in general - add a label for the data format if available Type: Bug Issues related to software defects or unexpected behavior, which require resolution.
Milestone

Comments

@jquartel
Copy link

I found on one of my projects that OR was trimming trailing white space from some strings when my TSV was imported, even though the 'Trim leading & trailing whitespace from strings' checkbox was left off.

Here is a simple TSV I made to illustrate the point. The entries " lsp" have a leading space, "tsp " have a trailing space, and " ltsp " have both a leading and a trailing space (in all cases a single space character). Import this into OR (I am using 3.8.0 on MacOS 14.4.1) and you find that all entries have lost their spaces. I expect this to only happen if the 'Trim leading & trailing whitespace from strings' option is checked.
Spaces.tsv.zip

@jquartel jquartel added Status: Pending Review Indicates that the issue or pull request is awaiting review by project maintainers or collaborators Type: Bug Issues related to software defects or unexpected behavior, which require resolution. labels May 14, 2024
@tfmorris tfmorris added import About importers in general - add a label for the data format if available CSV/TSV About the CSV/TSV import or export and removed Status: Pending Review Indicates that the issue or pull request is awaiting review by project maintainers or collaborators labels May 14, 2024
@tfmorris tfmorris self-assigned this May 14, 2024
@tfmorris tfmorris added this to the 3.8 milestone May 15, 2024
tfmorris added a commit to tfmorris/OpenRefine that referenced this issue May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSV/TSV About the CSV/TSV import or export import About importers in general - add a label for the data format if available Type: Bug Issues related to software defects or unexpected behavior, which require resolution.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants