Skip to content

Re-ran tutorial to remove errors#2925

Merged
Adel-Moumen merged 1 commit into
speechbrain:developfrom
pplantinga:fix-sc-tutorial
Jun 3, 2025
Merged

Re-ran tutorial to remove errors#2925
Adel-Moumen merged 1 commit into
speechbrain:developfrom
pplantinga:fix-sc-tutorial

Conversation

@pplantinga

@pplantinga pplantinga commented May 22, 2025

Copy link
Copy Markdown
Collaborator

Fixes #2924

Basically just reran the tutorial on colab, everything still works I guess.

The one thing this revealed that we might want to fix later is that the template now spits out a bunch of:

speechbrain.dataio.encoder - CategoricalEncoder.expect_len was never called: assuming category count of 28 to be correct! Sanity check your encoder using `.expect_len`. Ensure that downstream code also uses the correct size. If you are sure this does not apply to you, use `.ignore_len`.

You can use the readthedocs build link in the CI checks or this one:

https://speechbrain--2925.org.readthedocs.build/en/2925/tutorials/tasks/speech-classification-from-scratch.html

@Adel-Moumen

Adel-Moumen commented May 23, 2025

Copy link
Copy Markdown
Collaborator

The one thing this revealed that we might want to fix later is that the template now spits out a bunch of:

I think this is an issue that we might want to address here as well. I knew we had a bunch of this popping, but I didn't thought it was every epochs... I can have a deeper look.

@pplantinga

Copy link
Copy Markdown
Collaborator Author

The one thing this revealed that we might want to fix later is that the template now spits out a bunch of:

I think this is an issue that we might want to address here as well. I knew we had a bunch of this popping, but I didn't thought it was every epochs... I can have a deeper look.

Perhaps just switching to warnings.warn could be enough to reduce warnings to one

@Adel-Moumen Adel-Moumen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Adel-Moumen Adel-Moumen merged commit 6916175 into speechbrain:develop Jun 3, 2025
5 checks passed
@pplantinga pplantinga deleted the fix-sc-tutorial branch October 31, 2025 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Errors exist in Speech Classification From Scratch Tutorial

2 participants