Skip to content
This repository has been archived by the owner on Oct 30, 2023. It is now read-only.

Update tfdata.shuffle buffer_size #1

Merged
merged 1 commit into from Nov 4, 2021
Merged

Update tfdata.shuffle buffer_size #1

merged 1 commit into from Nov 4, 2021

Conversation

MatinHz
Copy link
Member

@MatinHz MatinHz commented Nov 4, 2021

From the TF documentation: "For perfect shuffling, a buffer size greater than or equal to the full size of the dataset is required."
https://github.com/tensorflow/tensorflow/blob/3aa40c3ce9d16eae296f086bc4ac4d62deb2affc/tensorflow/python/data/ops/dataset_ops.py#L1300

From the TF documentation: "For perfect shuffling, a buffer size greater than or equal to the full size of the dataset is required."
https://github.com/tensorflow/tensorflow/blob/3aa40c3ce9d16eae296f086bc4ac4d62deb2affc/tensorflow/python/data/ops/dataset_ops.py#L1300
@MatinHz MatinHz requested a review from anindox8 November 4, 2021 11:40
@anindox8
Copy link
Member

anindox8 commented Nov 4, 2021

For ProstateX, it's fine. But for larger datasets, this may not be feasible. Nonetheless, it's flexible and can always be changed and as of now, 0.50 is really arbitrary. So I'll merge it in. Thanks!

@anindox8 anindox8 merged commit d6eaf0e into main Nov 4, 2021
@anindox8 anindox8 deleted the MatinHz-patch-1 branch February 11, 2022 14:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants