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

PR: Add Whitespace option in the import wizard #4583

Merged
merged 1 commit into from
Jun 11, 2017

Conversation

bcolsen
Copy link
Member

@bcolsen bcolsen commented Jun 10, 2017

This PR fixes #4581 by adding continuous whitespace as an option to the column separator section of the import wizard.

Let me know if this needs a test, and I'll write one.

Currently the data import wizard doesn't play nicely with text data with multiple spaces as delimiters(or maybe I don't play nicely with the data importer ;-). This is common in fixed width data sets like this:

FIRST  LAST           STATE     TELEPHONE      AGE   WEIGHT
John   Smith          WA        418-Y11-4111    26    3.0456
Mary   Hartford       CA        319-Z19-4341     5   13.5687
Evan   Nolan          IL        219-532-c301   145   45.7890

Putting a ' ' (a space) in a the 'other' text box gives this:

screenshot from 2017-06-09 22-54-23

When it could give this:

screenshot from 2017-06-09 22-59-21

It looks like this:

image

@ccordoba12 ccordoba12 added this to the v3.2 milestone Jun 11, 2017
@ccordoba12 ccordoba12 changed the title Added Whitespace option in the import wizard PR: Add Whitespace option in the import wizard Jun 11, 2017
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@ccordoba12 ccordoba12 merged commit 89603c9 into spyder-ide:3.x Jun 11, 2017
ccordoba12 added a commit that referenced this pull request Jun 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants