NeMo icon indicating copy to clipboard operation
NeMo copied to clipboard

IA3 support for GPT and T5

Open arendu opened this issue 2 years ago • 6 comments

Signed-off-by: arendu [email protected]

What does this PR do ?

Add a one line overview of what this PR aims to accomplish.

Collection: [Note which collection this PR will affect]

Changelog

  • Add specific line by line info of high level changes in this PR.

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)

arendu avatar Sep 09 '22 06:09 arendu

This pull request introduces 1 alert when merging bc278d271f183f50471c9d4eac92d3fc8b66150b into 7357d4b107c821ff9f890ae1b9d9d0b9fe207890 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 09 '22 07:09 lgtm-com[bot]

This pull request introduces 1 alert when merging eaf620aca0d4987a6d579b0507a92e0963b1a3c6 into 3e23f9062818f63a629ec0a59f6caa9bed3f28fb - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 12 '22 22:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 96ef74674d384077052cec7407a6125260bd7030 into 4f6aba138173730eef04b11f6456fa7b12d6075d - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 12 '22 22:09 lgtm-com[bot]

This pull request introduces 3 alerts when merging 2c8d88b4bf3ca709ac75b46dde7404bab28c6f05 into 5a79c96f335753aba9f73b6912a86313aa8f2c9c - view on LGTM.com

new alerts:

  • 3 for Unused import

lgtm-com[bot] avatar Sep 13 '22 22:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 9d9cff4ec7bbc105ed8d3e1591bcec674d415a5b into f1825bc4b724b78c2d6ca392b616e8dc9a8cde04 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 19 '22 18:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 8f05db95a5b360ea4b25ddc3c7830e3e82049ef5 into a0d3727276f1ab23f651cbefbe142b7c632eee4b - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 20 '22 23:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 36b5f537346a104faf0ec0e5e910d54f7472d5e1 into 165112d3543d8945ce894fa0c99f206c0c6d3276 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 21 '22 19:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 90fb5be00de9ef96c3523faad074dec844818e51 into 9ada1415571add8d24572039e79d386cb7a5f4de - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 21 '22 21:09 lgtm-com[bot]

This pull request introduces 1 alert when merging d3d577032b40de4cb4eecf71a3b7376db11b1c7e into 8aab6e0941930e63769282a9ab856c2d42fb08a3 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 21 '22 21:09 lgtm-com[bot]

This pull request introduces 1 alert when merging f1780dcf98c8cddabbda1bb7c784933a02c355d3 into 576570be1db68cf8927e218985b294b9f95c6a37 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 23 '22 00:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 86f6f9277dba68e2c628c12b51a9eab4e1fdc00e into e872061accf2534a8b8f4542e05c5887bcfdecda - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 23 '22 18:09 lgtm-com[bot]

I have no issues with the pr wrr adapters, please have @vadam5 review the rest of it

titu1994 avatar Sep 29 '22 21:09 titu1994

This pull request introduces 1 alert when merging 282ab0d876823864cc141f8ef171d92958cf52a3 into be4c662500bf797c345bb0f4d4b17d251293f009 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 30 '22 21:09 lgtm-com[bot]

This pull request introduces 1 alert when merging 0b7aabd2f020d725e1d30d817e391c39063b9188 into be4c662500bf797c345bb0f4d4b17d251293f009 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Sep 30 '22 22:09 lgtm-com[bot]

This pull request introduces 1 alert when merging bf96cb0ae4260120320e5f1f11b44857a5eeb54a into 6d5a0d2df8555781181ffcd6d01a974c6e34aafc - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Oct 02 '22 05:10 lgtm-com[bot]

This pull request introduces 1 alert when merging 5a52810edf84837c35609dca3ca82e5ad940bf00 into bd1209cfc4a4646af64f78c67bb5401d06c14558 - view on LGTM.com

new alerts:

  • 1 for Unused import

lgtm-com[bot] avatar Oct 03 '22 16:10 lgtm-com[bot]