Patch for NMT accuracy regression from PTL 1.1.3 to PTL 1.1.4+
State: closed | Pull request: yes

The default for `find_unused_parameters` was changed from `True` to `False` in PTL. We're seeing higher accuracy in NMT with `find_unused_parameters=True`. Overriding this in NMT model until we learn more.

This pull request **introduces 1 alert** when merging 03dc67e4e06a7fd939ec32ae6a3ae2ed07371759 into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-c00b16affad2c7a4c82e0998d3d0995a058cd2fa)

**new alerts:**

* 1 for Unused import

This pull request **introduces 1 alert** when merging aef07cbe74e840895837e762d43cec01edf20520 into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-9cf8fbb6a2d4f0b68c50296c181384e3550def58)

**new alerts:**

* 1 for Unused import

This pull request **introduces 4 alerts** when merging 8c31f4a4477ff00015b8ae2090088037efdc4117 into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-3cd0f71bd1172d2ed2de884efa167a1af42e6667)

**new alerts:**

* 4 for Unused import

This pull request **introduces 6 alerts** when merging 02207f86415e790202848f2c36cec7b9c4d434d5 into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-c9c4ab115bd8a7f9e0c5fcea57199420d23b2096)

**new alerts:**

* 6 for Unused import

This pull request **introduces 3 alerts** when merging d5d4b232bad996a0d580cf60861da375c6e67d94 into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-8034262612377c60fb47429f0eaed321a4ed596c)

**new alerts:**

* 3 for Unused import

This pull request **introduces 6 alerts** when merging 8035fb988a2ea99060906d61e556c4c567f0783e into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-2a53970fd049a53b63fe0aecc48e2ebba93538d3)

**new alerts:**

* 6 for Unused import

This pull request **introduces 6 alerts** when merging 13e9523583dee6c3162f788f0ded35d40e8be195 into ad0edf2e6910b7b6de3ae93fba9b728f6479c50e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-8a488418abeec43276230b5d76f0919febc82bbb)

**new alerts:**

* 6 for Unused import

This pull request **introduces 7 alerts** when merging c8cab3bb299ec160506d29049d535b204461ce7a into 838892b1d38650bdf1bd829437f5722589f3ae42 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-0e82ea57b3b8d1489dc063ca5698a74d96827759)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging d140ecb485c0e6ee39187dbde63c78c7c6b578ff into c4379da172c1ac866fea0cd91ac6433dcd119651 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-cd9d7dcd746bf7bc876c7b0f2d4fdd66b0034e4d)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging 4f612b8e057aad68b38eb0640de1fab9ee892412 into 4fe96b152a74ba2dd3e044cd76fe91dfda566c08 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-be0ac3f1ed9da68c4be76cfc657b8496201aaa83)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging 26e7446e098ed675511db01582d6d0aabd0bedc0 into 3a67cd013c451f65eba75a0c54dcdcad89c9e95e - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-ddc1888fd8d6a12ea682855215b9eb37b49fe0ce)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging 1e1e08a9c0e991ccf20bce8e71d9b394065aa068 into 4fc670ad0c886be2623247921d4311ba30f486f8 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-6201fb13723cee2bb0b099feab30fc972ec9dc45)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging 1af04e4192fbf8b240cbc8a7e6924619b2479397 into 4fc670ad0c886be2623247921d4311ba30f486f8 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-ed7ec377da0e2e12033f12e9d30436abf85bb56b)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging 31b2529424ef89aa61cd21bd24420539ce5bac84 into 4fc670ad0c886be2623247921d4311ba30f486f8 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-65d9475e9b65afc5f4b6358083158f4bc2d5bc0d)

**new alerts:**

* 7 for Unused import

This pull request **introduces 7 alerts** when merging 20a37cec79729323ba976a0905e37daa019ea1f9 into 4fc670ad0c886be2623247921d4311ba30f486f8 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-3b89e3a9955ae19480c4127dd192c30b4c4d4048)

**new alerts:**

* 7 for Unused import

This pull request **fixes 2 alerts** when merging 0552327ff2bb3f84eec23a9643a50c8c9b3525f8 into 4fc670ad0c886be2623247921d4311ba30f486f8 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-c5bcb22771d638b0df7c818b0e55893454023d6b)

**fixed alerts:**

* 2 for Unused import

This pull request **fixes 2 alerts** when merging 6da45cfd3132804ede19bebde0e22479b3640619 into 6c34fe9d44d0c398dd9a87404f9fa45616a13647 - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-50cdc7a69a1c92dbc3d8880909231c3d36f5252d)

**fixed alerts:**

* 2 for Unused import

This pull request **fixes 2 alerts** when merging 4e3fdaac7c945785149d74d74b0f7a41cdc03451 into 3c9e60ab269e858cc73e780e5f85f749a8b8dcff - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-03fbce0110733279aa8078fea9fdb5858ebd1f55)

**fixed alerts:**

* 2 for Unused import

This pull request **fixes 2 alerts** when merging a6f200f29fe688b91db0cf82565542d1fcd5cb50 into 090cfae5db9b6f9eb62b3ede37ed438875a150df - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-c28ba7668fcf2c5ae9052379e659cd92127064e0)

**fixed alerts:**

* 2 for Unused import

This pull request **fixes 2 alerts** when merging 65e1b3f20fa063206b8cdd4ff043572a19f55fa3 into 090cfae5db9b6f9eb62b3ede37ed438875a150df - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-bc0906d35565b3d2304aa6c1589223d1e1e5b2c0)

**fixed alerts:**

* 2 for Unused import

This pull request **fixes 2 alerts** when merging 368d32bfbbe945426ce30025692506a80061e3ca into 4f87abb242fee7f0131fd27a266883c8740d8fac - [view on LGTM.com](https://lgtm.com/projects/g/NVIDIA/NeMo/rev/pr-d83fa003be981db281e39d1f6a282e677680386a)

**fixed alerts:**

* 2 for Unused import