Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix speed perturbation when not using transcript in slu.sh#5671

Merged
sw005320 merged 1 commit intoespnet:masterfrom
siddhu001:Fix_transformers_load
Feb 22, 2024
Merged

Fix speed perturbation when not using transcript in slu.sh#5671
sw005320 merged 1 commit intoespnet:masterfrom
siddhu001:Fix_transformers_load

Conversation

@siddhu001
Copy link
Collaborator

What?

I updated the "utt_extra_files" to be set as none (i.e., "") when not utilizing the transcript.

Why?

This caused some issues when performing speed perturbation without utilizing the transcript.

See also

#5656

@mergify mergify bot added the ESPnet2 label Feb 22, 2024
@siddhu001 siddhu001 mentioned this pull request Feb 22, 2024
@codecov
Copy link

codecov bot commented Feb 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8ca64cb) 76.12% compared to head (4a7b0e0) 76.12%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5671   +/-   ##
=======================================
  Coverage   76.12%   76.12%           
=======================================
  Files         744      744           
  Lines       69250    69250           
=======================================
  Hits        52715    52715           
  Misses      16535    16535           
Flag Coverage Δ
test_configuration_espnet2 ∅ <ø> (∅)
test_integration_espnet1 62.92% <ø> (ø)
test_integration_espnet2 49.00% <ø> (ø)
test_python_espnet1 18.36% <ø> (ø)
test_python_espnet2 52.64% <ø> (ø)
test_utils 22.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sw005320 sw005320 added Bugfix SLU Spoken language understanding labels Feb 22, 2024
@sw005320 sw005320 added this to the v.202405 milestone Feb 22, 2024
@sw005320 sw005320 merged commit 9718190 into espnet:master Feb 22, 2024
@sw005320
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix ESPnet2 SLU Spoken language understanding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants