Skip to content

Commit

Permalink
Fix the silly bug of f'{None}' prints 'None'. Huge thanks, Ethan!
Browse files Browse the repository at this point in the history
  • Loading branch information
lukewys committed Nov 18, 2021
1 parent 70def03 commit bb6aed5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ddsp/training/data.py
Original file line number Diff line number Diff line change
Expand Up @@ -426,7 +426,7 @@ def __init__(self,
split: Choices include ['train', 'test'].
suffix: Choices include [None, 'batched', 'unbatched'], but broadly
applies to any suffix adding to the file pattern.
When suffix is not None, will add "_suffix" to the file pattern.
When suffix is not None, will add "_{suffix}" to the file pattern.
This option is used in gs://magentadata/datasets/urmp/urmp_20210324.
With the "batched" suffix, the dataloader will load tfrecords
containing segmented audio samples in 4 seconds. With the "unbatched"
Expand All @@ -437,7 +437,7 @@ def __init__(self,
self.instrument_key = instrument_key
self.split = split
self.base_dir = base_dir
self.suffix = suffix if suffix is None else '_' + suffix
self.suffix = '' if suffix is None else '_' + suffix
super().__init__()

@property
Expand Down

0 comments on commit bb6aed5

Please sign in to comment.