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

owpreprocess: Add choice in FilteringModule for All Files (*) #334

Merged
merged 1 commit into from
Jan 28, 2018
Merged

owpreprocess: Add choice in FilteringModule for All Files (*) #334

merged 1 commit into from
Jan 28, 2018

Conversation

chasets
Copy link
Contributor

@chasets chasets commented Jan 27, 2018

Issue

The FileWidget used in the FilteringModule class of owpreprocess allows opening files with a .txt extension only. The stopwords files included with the nltk_data do not have this extension. So these files nor any other files lacking a .txt extension can be selected as files for Stopwords or Lexicons.

Fixes #333.

Description of changes

This change adds a selector for "All files" in addition to "Only text files" in the FileWidget creation for the FilteringModule class.

all_files_choice

The documentation describes creating files from scratch with a .txt extension, which is still appropriate, so no documentation changes are needed. No tests cover the FileWidget dialogs used by the FilteringModule.

Includes
  • Code changes
  • Tests
  • Documentation

The FileWidget used in the FilteringModule class of owpreprocess only allows opening
files with a .txt extension. The stopwords files included with the nltk_data do not
have this extension. This change adds a selector for "All Files (*)" in addition to
"Only text files (*.txt)" in the FileWidget.
@kernc
Copy link
Contributor

kernc commented Jan 28, 2018

Excellent in all respects! The Travis error, I'll wager, is unrelated. Merging.

Thanks!

@kernc kernc merged commit a85b4f7 into biolab:master Jan 28, 2018
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