Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PostprocessingDataset #1596

Merged
merged 73 commits into from
Jul 30, 2024
Merged

PostprocessingDataset #1596

merged 73 commits into from
Jul 30, 2024

Conversation

NeoLegends
Copy link
Collaborator

@NeoLegends NeoLegends commented Jul 25, 2024 •
edited
Loading

Closes #1505

@NeoLegends NeoLegends self-assigned this Jul 25, 2024
@albertz albertz changed the title Implement generic postprocessing dataset PostprocessingDataset Jul 25, 2024
@albertz

This comment was marked as resolved.

Sign in to view
@albertz

This comment was marked as resolved.

Sign in to view
@albertz

This comment was marked as resolved.

Sign in to view
@NeoLegends

This comment was marked as resolved.

Sign in to view
NeoLegends
NeoLegends commented Jul 26, 2024
View reviewed changes
returnn/datasets/audio.py Outdated Show resolved Hide resolved
returnn/datasets/postprocessing.py Outdated Show resolved Hide resolved
returnn/datasets/postprocessing.py Outdated Show resolved Hide resolved
albertz added a commit that referenced this pull request Jul 30, 2024
@NeoLegends @albertz
OggZipDataset various fixes (#1599)
61ad52a 
* No longer relies on init_seq_order for data shape/type accessors to be callable
* No longer output dummy data w/ wrong shape
* No longer define num_outputs["data"] when no features are extracted.

Pulled out from #1596

Co-authored-by: Albert Zeyer <albzey@gmail.com>
NeoLegends and others added 2 commits July 30, 2024 14:05
@NeoLegends @albertz
Update returnn/datasets/postprocessing.py
0383614 
Co-authored-by: Albert Zeyer <albzey@gmail.com>
@NeoLegends @albertz
Update returnn/datasets/postprocessing.py
70b2fbc 
Co-authored-by: Albert Zeyer <albzey@gmail.com>
@NeoLegends NeoLegends requested a review from albertz July 30, 2024 13:12
@albertz albertz merged commit 7b46160 into master Jul 30, 2024
63 checks passed
@albertz albertz deleted the moritz-dataset-pp branch July 30, 2024 13:37
@albertz albertz mentioned this pull request Jul 30, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@albertz albertz albertz approved these changes

Assignees

@NeoLegends NeoLegends

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Dataset post processing
2 participants
@NeoLegends @albertz