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

Add kwargs to exact string match #4479

Merged
merged 1 commit into from
Jun 30, 2022

Conversation

MaximumEntropy
Copy link
Contributor

Signed-off-by: MaximumEntropy [email protected]

What does this PR do ?

Exact string match fix for finetuning.

Collection: NLP

Changelog

  • Exact string match fix for fine-tuning by adding kwargs.

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this 

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

PR Type:

  • New Feature
  • Bugfix
  • Documentation

If you haven't finished some of the above items you can still open "Draft" PR.

Who can review?

Anyone in the community is free to review the PR once the checks have passed.
Contributor guidelines contains specific people who can review PRs to various areas.

Additional Information

  • Related to # (issue)

Signed-off-by: MaximumEntropy <[email protected]>
Copy link
Collaborator

@ericharper ericharper left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@MaximumEntropy MaximumEntropy merged commit a8f5f63 into r1.10.0 Jun 30, 2022
@MaximumEntropy MaximumEntropy deleted the exact_string_match_metric_fix branch June 30, 2022 22:06
ericharper pushed a commit that referenced this pull request Jul 1, 2022
ericharper added a commit that referenced this pull request Jul 1, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix ASR Typos in tutorials (#4384)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Quick wav2vec fix. In-place operation adding convolutional positions to encoder was overwriting leaf history. Wasn't caught on previous torch versions. (#4383)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: tbartley94 <[email protected]>
(cherry picked from commit 0322b15)

Co-authored-by: Travis Bartley <[email protected]>

* Fix tutorial typos and docs (#4415)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Add ASR Scores to Docs (#4412)

* Fix link

Signed-off-by: smajumdar <[email protected]>

* Correct model card

Signed-off-by: smajumdar <[email protected]>

* Add ASR Results to Docs

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* docs: add table overflow handling for nested sections (#4441)

Co-authored-by: Nick Goncharenko <[email protected]>

* Docs: Decrease Font Size on Tables  (#4444)

* docs: add table overflow handling for nested sections

* docs: set table font-size to small

Co-authored-by: Nick Goncharenko <[email protected]>

* Updated notebook to fix batch configuration and precision bugs (#4447)

* Updated notebook to fix batch configuration and precision bugs

Signed-off-by: Virginia Adams <[email protected]>

* Deleted cell outputs

Signed-off-by: Virginia Adams <[email protected]>

* Set datasets back to full dataset

Signed-off-by: Virginia Adams <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* fix branch in link (#4454)

Signed-off-by: ekmb <[email protected]>

* [TTS] [bugfix] German FastPitch HiFi-GAN tutorial and lr (#4459)

* [TN] Bug fix: expand serial coverage of unknown symbol, remove constraints from word graph (#4463)

* remove constraints from word graph det

Signed-off-by: ekmb <[email protected]>

* add measure units to serial

Signed-off-by: ekmb <[email protected]>

* revert serial changes, update jenkins path

Signed-off-by: ekmb <[email protected]>

* fix test case

Signed-off-by: ekmb <[email protected]>

* update indentation (#4468)

Signed-off-by: Akshit Arora <[email protected]>

* t5-rpe-fix targeting r1.10.0; raise exception for PP>2. (#4469)

Signed-off-by: Hoo Chang Shin <[email protected]>

Co-authored-by: Hoo Chang Shin <[email protected]>

* Fix some 's' cases for IPA G2P (#4460)

Signed-off-by: Jocelyn Huang <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* Refactor bias act fusion (#4376)

* Refactor bias act fusion

Signed-off-by: MaximumEntropy <[email protected]>

* Update NMT config

Signed-off-by: MaximumEntropy <[email protected]>

* Update ci tests

Signed-off-by: MaximumEntropy <[email protected]>

* Empty

Signed-off-by: MaximumEntropy <[email protected]>

* Add kwargs to exact string match (#4479)

Signed-off-by: MaximumEntropy <[email protected]>

* Try fix (#4484)

Signed-off-by: MaximumEntropy <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Somshubra Majumdar <[email protected]>
Co-authored-by: Travis Bartley <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Virginia Adams <[email protected]>
Co-authored-by: Evelina <[email protected]>
Co-authored-by: Akshit Arora <[email protected]>
Co-authored-by: khcs <[email protected]>
Co-authored-by: Hoo Chang Shin <[email protected]>
Co-authored-by: Jocelyn <[email protected]>
Co-authored-by: Sandeep Subramanian <[email protected]>
XuesongYang pushed a commit that referenced this pull request Jul 5, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix ASR Typos in tutorials (#4384)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Quick wav2vec fix. In-place operation adding convolutional positions to encoder was overwriting leaf history. Wasn't caught on previous torch versions. (#4383)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: tbartley94 <[email protected]>
(cherry picked from commit 0322b15)

Co-authored-by: Travis Bartley <[email protected]>

* Fix tutorial typos and docs (#4415)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Add ASR Scores to Docs (#4412)

* Fix link

Signed-off-by: smajumdar <[email protected]>

* Correct model card

Signed-off-by: smajumdar <[email protected]>

* Add ASR Results to Docs

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* docs: add table overflow handling for nested sections (#4441)

Co-authored-by: Nick Goncharenko <[email protected]>

* Docs: Decrease Font Size on Tables  (#4444)

* docs: add table overflow handling for nested sections

* docs: set table font-size to small

Co-authored-by: Nick Goncharenko <[email protected]>

* Updated notebook to fix batch configuration and precision bugs (#4447)

* Updated notebook to fix batch configuration and precision bugs

Signed-off-by: Virginia Adams <[email protected]>

* Deleted cell outputs

Signed-off-by: Virginia Adams <[email protected]>

* Set datasets back to full dataset

Signed-off-by: Virginia Adams <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* fix branch in link (#4454)

Signed-off-by: ekmb <[email protected]>

* [TTS] [bugfix] German FastPitch HiFi-GAN tutorial and lr (#4459)

* [TN] Bug fix: expand serial coverage of unknown symbol, remove constraints from word graph (#4463)

* remove constraints from word graph det

Signed-off-by: ekmb <[email protected]>

* add measure units to serial

Signed-off-by: ekmb <[email protected]>

* revert serial changes, update jenkins path

Signed-off-by: ekmb <[email protected]>

* fix test case

Signed-off-by: ekmb <[email protected]>

* update indentation (#4468)

Signed-off-by: Akshit Arora <[email protected]>

* t5-rpe-fix targeting r1.10.0; raise exception for PP>2. (#4469)

Signed-off-by: Hoo Chang Shin <[email protected]>

Co-authored-by: Hoo Chang Shin <[email protected]>

* Fix some 's' cases for IPA G2P (#4460)

Signed-off-by: Jocelyn Huang <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* Refactor bias act fusion (#4376)

* Refactor bias act fusion

Signed-off-by: MaximumEntropy <[email protected]>

* Update NMT config

Signed-off-by: MaximumEntropy <[email protected]>

* Update ci tests

Signed-off-by: MaximumEntropy <[email protected]>

* Empty

Signed-off-by: MaximumEntropy <[email protected]>

* Add kwargs to exact string match (#4479)

Signed-off-by: MaximumEntropy <[email protected]>

* Try fix (#4484)

Signed-off-by: MaximumEntropy <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Somshubra Majumdar <[email protected]>
Co-authored-by: Travis Bartley <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Virginia Adams <[email protected]>
Co-authored-by: Evelina <[email protected]>
Co-authored-by: Akshit Arora <[email protected]>
Co-authored-by: khcs <[email protected]>
Co-authored-by: Hoo Chang Shin <[email protected]>
Co-authored-by: Jocelyn <[email protected]>
Co-authored-by: Sandeep Subramanian <[email protected]>
arendu pushed a commit that referenced this pull request Jul 21, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix ASR Typos in tutorials (#4384)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Quick wav2vec fix. In-place operation adding convolutional positions to encoder was overwriting leaf history. Wasn't caught on previous torch versions. (#4383)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: tbartley94 <[email protected]>
(cherry picked from commit 0322b15)

Co-authored-by: Travis Bartley <[email protected]>

* Fix tutorial typos and docs (#4415)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Add ASR Scores to Docs (#4412)

* Fix link

Signed-off-by: smajumdar <[email protected]>

* Correct model card

Signed-off-by: smajumdar <[email protected]>

* Add ASR Results to Docs

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* docs: add table overflow handling for nested sections (#4441)

Co-authored-by: Nick Goncharenko <[email protected]>

* Docs: Decrease Font Size on Tables  (#4444)

* docs: add table overflow handling for nested sections

* docs: set table font-size to small

Co-authored-by: Nick Goncharenko <[email protected]>

* Updated notebook to fix batch configuration and precision bugs (#4447)

* Updated notebook to fix batch configuration and precision bugs

Signed-off-by: Virginia Adams <[email protected]>

* Deleted cell outputs

Signed-off-by: Virginia Adams <[email protected]>

* Set datasets back to full dataset

Signed-off-by: Virginia Adams <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* fix branch in link (#4454)

Signed-off-by: ekmb <[email protected]>

* [TTS] [bugfix] German FastPitch HiFi-GAN tutorial and lr (#4459)

* [TN] Bug fix: expand serial coverage of unknown symbol, remove constraints from word graph (#4463)

* remove constraints from word graph det

Signed-off-by: ekmb <[email protected]>

* add measure units to serial

Signed-off-by: ekmb <[email protected]>

* revert serial changes, update jenkins path

Signed-off-by: ekmb <[email protected]>

* fix test case

Signed-off-by: ekmb <[email protected]>

* update indentation (#4468)

Signed-off-by: Akshit Arora <[email protected]>

* t5-rpe-fix targeting r1.10.0; raise exception for PP>2. (#4469)

Signed-off-by: Hoo Chang Shin <[email protected]>

Co-authored-by: Hoo Chang Shin <[email protected]>

* Fix some 's' cases for IPA G2P (#4460)

Signed-off-by: Jocelyn Huang <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* Refactor bias act fusion (#4376)

* Refactor bias act fusion

Signed-off-by: MaximumEntropy <[email protected]>

* Update NMT config

Signed-off-by: MaximumEntropy <[email protected]>

* Update ci tests

Signed-off-by: MaximumEntropy <[email protected]>

* Empty

Signed-off-by: MaximumEntropy <[email protected]>

* Add kwargs to exact string match (#4479)

Signed-off-by: MaximumEntropy <[email protected]>

* Try fix (#4484)

Signed-off-by: MaximumEntropy <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Somshubra Majumdar <[email protected]>
Co-authored-by: Travis Bartley <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Virginia Adams <[email protected]>
Co-authored-by: Evelina <[email protected]>
Co-authored-by: Akshit Arora <[email protected]>
Co-authored-by: khcs <[email protected]>
Co-authored-by: Hoo Chang Shin <[email protected]>
Co-authored-by: Jocelyn <[email protected]>
Co-authored-by: Sandeep Subramanian <[email protected]>
Signed-off-by: arendu <[email protected]>
Davood-M pushed a commit to Davood-M/NeMo that referenced this pull request Aug 9, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix ASR Typos in tutorials (NVIDIA#4384)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Quick wav2vec fix. In-place operation adding convolutional positions to encoder was overwriting leaf history. Wasn't caught on previous torch versions. (NVIDIA#4383)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: tbartley94 <[email protected]>
(cherry picked from commit 0322b15)

Co-authored-by: Travis Bartley <[email protected]>

* Fix tutorial typos and docs (NVIDIA#4415)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Add ASR Scores to Docs (NVIDIA#4412)

* Fix link

Signed-off-by: smajumdar <[email protected]>

* Correct model card

Signed-off-by: smajumdar <[email protected]>

* Add ASR Results to Docs

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* docs: add table overflow handling for nested sections (NVIDIA#4441)

Co-authored-by: Nick Goncharenko <[email protected]>

* Docs: Decrease Font Size on Tables  (NVIDIA#4444)

* docs: add table overflow handling for nested sections

* docs: set table font-size to small

Co-authored-by: Nick Goncharenko <[email protected]>

* Updated notebook to fix batch configuration and precision bugs (NVIDIA#4447)

* Updated notebook to fix batch configuration and precision bugs

Signed-off-by: Virginia Adams <[email protected]>

* Deleted cell outputs

Signed-off-by: Virginia Adams <[email protected]>

* Set datasets back to full dataset

Signed-off-by: Virginia Adams <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* fix branch in link (NVIDIA#4454)

Signed-off-by: ekmb <[email protected]>

* [TTS] [bugfix] German FastPitch HiFi-GAN tutorial and lr (NVIDIA#4459)

* [TN] Bug fix: expand serial coverage of unknown symbol, remove constraints from word graph (NVIDIA#4463)

* remove constraints from word graph det

Signed-off-by: ekmb <[email protected]>

* add measure units to serial

Signed-off-by: ekmb <[email protected]>

* revert serial changes, update jenkins path

Signed-off-by: ekmb <[email protected]>

* fix test case

Signed-off-by: ekmb <[email protected]>

* update indentation (NVIDIA#4468)

Signed-off-by: Akshit Arora <[email protected]>

* t5-rpe-fix targeting r1.10.0; raise exception for PP>2. (NVIDIA#4469)

Signed-off-by: Hoo Chang Shin <[email protected]>

Co-authored-by: Hoo Chang Shin <[email protected]>

* Fix some 's' cases for IPA G2P (NVIDIA#4460)

Signed-off-by: Jocelyn Huang <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* Refactor bias act fusion (NVIDIA#4376)

* Refactor bias act fusion

Signed-off-by: MaximumEntropy <[email protected]>

* Update NMT config

Signed-off-by: MaximumEntropy <[email protected]>

* Update ci tests

Signed-off-by: MaximumEntropy <[email protected]>

* Empty

Signed-off-by: MaximumEntropy <[email protected]>

* Add kwargs to exact string match (NVIDIA#4479)

Signed-off-by: MaximumEntropy <[email protected]>

* Try fix (NVIDIA#4484)

Signed-off-by: MaximumEntropy <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Somshubra Majumdar <[email protected]>
Co-authored-by: Travis Bartley <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Virginia Adams <[email protected]>
Co-authored-by: Evelina <[email protected]>
Co-authored-by: Akshit Arora <[email protected]>
Co-authored-by: khcs <[email protected]>
Co-authored-by: Hoo Chang Shin <[email protected]>
Co-authored-by: Jocelyn <[email protected]>
Co-authored-by: Sandeep Subramanian <[email protected]>
Signed-off-by: David Mosallanezhad <[email protected]>
hainan-xv pushed a commit to hainan-xv/NeMo that referenced this pull request Nov 29, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix ASR Typos in tutorials (NVIDIA#4384)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Quick wav2vec fix. In-place operation adding convolutional positions to encoder was overwriting leaf history. Wasn't caught on previous torch versions. (NVIDIA#4383)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: tbartley94 <[email protected]>
(cherry picked from commit 0322b15)

Co-authored-by: Travis Bartley <[email protected]>

* Fix tutorial typos and docs (NVIDIA#4415)

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Fix typos

Signed-off-by: smajumdar <[email protected]>

* Add ASR Scores to Docs (NVIDIA#4412)

* Fix link

Signed-off-by: smajumdar <[email protected]>

* Correct model card

Signed-off-by: smajumdar <[email protected]>

* Add ASR Results to Docs

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* Update info

Signed-off-by: smajumdar <[email protected]>

* docs: add table overflow handling for nested sections (NVIDIA#4441)

Co-authored-by: Nick Goncharenko <[email protected]>

* Docs: Decrease Font Size on Tables  (NVIDIA#4444)

* docs: add table overflow handling for nested sections

* docs: set table font-size to small

Co-authored-by: Nick Goncharenko <[email protected]>

* Updated notebook to fix batch configuration and precision bugs (NVIDIA#4447)

* Updated notebook to fix batch configuration and precision bugs

Signed-off-by: Virginia Adams <[email protected]>

* Deleted cell outputs

Signed-off-by: Virginia Adams <[email protected]>

* Set datasets back to full dataset

Signed-off-by: Virginia Adams <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* fix branch in link (NVIDIA#4454)

Signed-off-by: ekmb <[email protected]>

* [TTS] [bugfix] German FastPitch HiFi-GAN tutorial and lr (NVIDIA#4459)

* [TN] Bug fix: expand serial coverage of unknown symbol, remove constraints from word graph (NVIDIA#4463)

* remove constraints from word graph det

Signed-off-by: ekmb <[email protected]>

* add measure units to serial

Signed-off-by: ekmb <[email protected]>

* revert serial changes, update jenkins path

Signed-off-by: ekmb <[email protected]>

* fix test case

Signed-off-by: ekmb <[email protected]>

* update indentation (NVIDIA#4468)

Signed-off-by: Akshit Arora <[email protected]>

* t5-rpe-fix targeting r1.10.0; raise exception for PP>2. (NVIDIA#4469)

Signed-off-by: Hoo Chang Shin <[email protected]>

Co-authored-by: Hoo Chang Shin <[email protected]>

* Fix some 's' cases for IPA G2P (NVIDIA#4460)

Signed-off-by: Jocelyn Huang <[email protected]>

Co-authored-by: Eric Harper <[email protected]>

* Refactor bias act fusion (NVIDIA#4376)

* Refactor bias act fusion

Signed-off-by: MaximumEntropy <[email protected]>

* Update NMT config

Signed-off-by: MaximumEntropy <[email protected]>

* Update ci tests

Signed-off-by: MaximumEntropy <[email protected]>

* Empty

Signed-off-by: MaximumEntropy <[email protected]>

* Add kwargs to exact string match (NVIDIA#4479)

Signed-off-by: MaximumEntropy <[email protected]>

* Try fix (NVIDIA#4484)

Signed-off-by: MaximumEntropy <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Somshubra Majumdar <[email protected]>
Co-authored-by: Travis Bartley <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Nick Goncharenko <[email protected]>
Co-authored-by: Virginia Adams <[email protected]>
Co-authored-by: Evelina <[email protected]>
Co-authored-by: Akshit Arora <[email protected]>
Co-authored-by: khcs <[email protected]>
Co-authored-by: Hoo Chang Shin <[email protected]>
Co-authored-by: Jocelyn <[email protected]>
Co-authored-by: Sandeep Subramanian <[email protected]>
Signed-off-by: Hainan Xu <[email protected]>
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.

2 participants