search

espnet / espnet_model_zoo

ESPnet Model Zoo
Apache License 2.0
241 stars 39 forks source link

Update table.csv #41

Closed guoaoo closed 3 years ago

guoaoo commented 3 years ago

fix name

codecov-commenter commented 3 years ago

Codecov Report

Merging #41 (607007b) into master (00beed6) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 607007b differs from pull request most recent head 0f0da01. Consider uploading reports for the commit 0f0da01 to get more accurate results Impacted file tree graph

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   53.45%   53.45%           
=======================================
  Files           2        2           
  Lines         318      318           
=======================================
  Hits          170      170           
  Misses        148      148

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 00beed6...0f0da01. Read the comment docs.

sw005320 commented 3 years ago

Thanks!

@kan-bayashi, could you also fix your zenodo entries? The issue comes from your typo…

kan-bayashi commented 3 years ago

Sorry for my typo. @sw005320 This type is only a problem of tag name so zenodo entry is fine.

sw005320 commented 3 years ago

Yes, you’re right. It would be better to make it consistent (to avoid some confusions), so it would be great if you fix them ;)

kan-bayashi commented 3 years ago

Sorry, I could not understand your meaning. What should I do?

sw005320 commented 3 years ago

Oh, maybe I misunderstood. I though that you made some typo in the zenodo uploading stage. You don’t have to do anything then.

sw005320 commented 3 years ago

https://github.com/espnet/espnet/search?q=trasnformer Could you fix them?