tensorflow / tensorflow/datasets
Fails on Windows due to different path separators
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 4.6k
- Forks
- 1.6k
- Avg merge
- 3h 54m
- Merged PRs (30d)
- 1
Description
In windows, the 'train_test_paths' and 'validation_paths' are not completely removed from 'train_paths'.
'train_paths' has '\' as path separator
'train_test_paths' and 'validation_paths' have '/' as path separator
This results in train and validation sets not split as expected.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at tensorflow_datasets/audio/speech_commands.py around line 193 and inspect how train_paths is compared with train_test_paths and validation_paths on Windows. Reproduce the path-separator mismatch and verify that the resulting train, test, and validation sets are split as expected on Windows.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python, tensorflow
- Domain
- data
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100