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

fix: Check for None type custom dict #106

Merged

Conversation

mohnishparmar
Copy link
Contributor

Add checks for default empty dict and skip iterating over none

atomer-nvidia
atomer-nvidia previously approved these changes Nov 29, 2024
@atomer-nvidia atomer-nvidia changed the base branch from release/24.12 to release/2.18.0 November 29, 2024 06:50
@atomer-nvidia atomer-nvidia dismissed their stale review November 29, 2024 06:50

The base branch was changed.

@atomer-nvidia
Copy link
Contributor

Changed the target. It should be targeted to release/2.18.0.

@atomer-nvidia atomer-nvidia merged commit 6f7bd6b into nvidia-riva:release/2.18.0 Nov 29, 2024
rmittal-github pushed a commit that referenced this pull request Dec 13, 2024
* fix: Check for None type custom dict (#106)

fix: Check for None type custom dict

* Add DNT and custom translation support in NMT client (#108)

* changes made for custom translation, dnt phrases

* formatted

* minor fixes done

* changes made

* minor fix done

* changes done

* changes made

* changes made for dnt and custom translation

* .gitmodules changed

---------

Co-authored-by: Manisha Johnson <[email protected]>

* fix: undeclared variable (#109)

* Updating git SHA to point to TOT main

* Updating git SHA to point to TOT main

---------

Co-authored-by: mohnishparmar <[email protected]>
Co-authored-by: ydharavath <[email protected]>
Co-authored-by: Manisha Johnson <[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