Fix bugs in `filter_features` command line utility. by desilinguist · Pull Request #660 · EducationalTestingService/skll
added 4 commits
February 4, 2021 16:43- Convert any labels specified on the command line using `safe_float()` just like the labels that are read by all of the FeatureSet reader classes. This ensures that something like `-L 0 1` is interpreted correctly.
- Right now, the input and output file need to be specified _before_ any optional arguments which is unintuitive. This commit makes it such that the files also require flags but they are required.
- Add new tests to check that the label conversion works correctly. - Update existing tests to use the new command line argument flags.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters