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.

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

Update show_asr_result.sh #4943

Merged
merged 1 commit into from Feb 17, 2023
Merged

Update show_asr_result.sh #4943

merged 1 commit into from Feb 17, 2023

Conversation

kamo-naoyuki
Copy link
Collaborator

@kamo-naoyuki kamo-naoyuki commented Feb 17, 2023

Issue:

show_asr_result.sh expects the result in exp/*/*/score_*/result.txt. If the data set has more deep hierarchical directory structure e.g. data/foo/bar/train_set, it doesn't work properly.

Modify:

  • Change the default value of maxdepth to 10
  • Change the subsection name of the generated markdown file: ## $(basename ${expdir}) -> "## ${expdir}"
  • Bug fix of printing ${expdir}/*/*/score_wer/scoring/*.filt.sys: no such file or directory

@kamo-naoyuki kamo-naoyuki added the auto-merge Enable auto-merge label Feb 17, 2023
@mergify mergify bot added the ESPnet2 label Feb 17, 2023
@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Merging #4943 (dd85850) into master (9c7bde4) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4943      +/-   ##
==========================================
- Coverage   76.63%   76.63%   -0.01%     
==========================================
  Files         604      604              
  Lines       53934    53934              
==========================================
- Hits        41334    41330       -4     
- Misses      12600    12604       +4     
Flag Coverage Δ
test_integration_espnet1 66.33% <ø> (ø)
test_integration_espnet2 47.40% <ø> (-0.02%) ⬇️
test_python 66.55% <ø> (ø)
test_utils 23.35% <ø> (ø)

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

Impacted Files Coverage Δ
espnet2/train/trainer.py 76.40% <0.00%> (-0.94%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mergify mergify bot merged commit f696c7a into master Feb 17, 2023
@mergify mergify bot deleted the kamo-naoyuki-patch-1 branch February 17, 2023 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enable auto-merge ESPnet2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant