Skip to content

Commit

Permalink
Simplify ReadFromTFDS.
Browse files Browse the repository at this point in the history
PiperOrigin-RevId: 698813440
  • Loading branch information
fineguy authored and The TensorFlow Datasets Authors committed Dec 4, 2024
1 parent 3700745 commit 870d16a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
8 changes: 2 additions & 6 deletions tensorflow_datasets/core/beam_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
from typing import Any

from tensorflow_datasets.core import dataset_builder
from tensorflow_datasets.core import lazy_imports_lib
from tensorflow_datasets.core import naming
from tensorflow_datasets.core.utils import shard_utils
from tensorflow_datasets.core.utils.lazy_imports_utils import apache_beam as beam
Expand All @@ -30,15 +29,13 @@
]


@lazy_imports_lib.beam_ptransform_fn
def ReadFromTFDS( # pylint: disable=invalid-name
pipeline,
builder: dataset_builder.DatasetBuilder,
split: str,
workers_per_shard: int = 1,
**as_dataset_kwargs: Any,
):
"""Creates a beam pipeline yielding TFDS examples.
"""Creates a beam PCollection yielding TFDS examples.
Each dataset shard will be processed in parallel.
Expand All @@ -63,7 +60,6 @@ def ReadFromTFDS( # pylint: disable=invalid-name
examples will be used.
Args:
pipeline: beam pipeline (automatically set)
builder: Dataset builder to load
split: Split name to load (e.g. `train+test`, `train`)
workers_per_shard: number of workers that should read a shard in parallel.
Expand Down Expand Up @@ -132,7 +128,7 @@ def load_shard(file_instruction: shard_utils.FileInstruction): # pylint: disabl
value=len(file_instructions),
namespace='ReadFromTFDS',
)
return pipeline | beam.Create(file_instructions) | beam.FlatMap(load_shard)
return beam.Create(file_instructions) | beam.FlatMap(load_shard)


@functools.lru_cache(None)
Expand Down
2 changes: 0 additions & 2 deletions tensorflow_datasets/core/beam_utils_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@
# See the License for the specific language governing permissions and
# limitations under the License.

"""Tests for beam_utils."""

import os
import pathlib
from typing import Optional
Expand Down

0 comments on commit 870d16a

Please sign in to comment.