Datasets: two files of same name but different capitalization

[MareoRaft]

You have a file called Text.md and one called text.md in the same directory datasets/docs/api_docs/python/tfds/features/.

This is causing an issue in filesystems that ignore capitalization in filenames. (My filesystem overwrote one of the files with the other, and now git shows unstaged changes/modification.)

Could you resolve the conflict between the two files?

[rsepassi]

Thanks for reporting. We'll see if we can rename one of them. We develop on Linux (which generally has case-sensitive filesystems) so that's why we didn't spot it before.

[rsepassi]

Are you using Windows? Have you otherwise found Windows usage to be working well?

[MareoRaft]

No, I experienced the issue on macOS Mojave (with Apple's newer APFS drive formatting).

On my other laptop (macOS High Sierra), the drive is also formatted in APFS. It looks like it would also experience the same issue:

~/Desktop/temp % touch this.txt
~/Desktop/temp % touch This.txt
~/Desktop/temp % ls | grep txt
this.txt

Apple has supported both case-sensitive and case-insensitive versions of their older Mac OS Extended (Journaled) format for drives for a long time. (but disclaimer, I'm not an expert)

Sorry I can't give you any feedback about Windows. tf is awesome though!

[rsepassi]

Great, thank you for clarifying. Confirmed that this is an issue on MacOS
as well.

On Mon, Dec 31, 2018 at 8:23 PM Matthew Lancellotti <
[email protected]> wrote:

No, I experienced the issue on macOS Mojave (with Apple's newer APFS drive
formatting).

On my other laptop (macOS High Sierra), the drive is also formatted in
APFS. It looks like it would also experience the same issue:

~/Desktop/temp % touch this.txt
~/Desktop/temp % touch This.txt
~/Desktop/temp % ls | grep txt
this.txt

Apple has supported both case-sensitive and case-insensitive versions of
their older Mac OS Extended (Journaled) format for drives for a long
time. (but disclaimer, I'm not an expert)

Sorry I can't give you any feedback about Windows. tf is awesome though!

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/datasets/issues/8#issuecomment-450709020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEGW5c4zE-NJN89AlEcxV2SfZchwO6Oks5u-uK2gaJpZM4ZUy1l
.

[Conchylicultor]

This should be fixed by: https://github.com/tensorflow/datasets/commit/ba1bec65eb310b86a68234d3d80f1d0d95dbeb69

Please re-open if you still get the issue