Telugu TN: Cardinal Semiotic Class - #447
Conversation
4a71ee5 to
f918e66
Compare
|
This PR is stale because it has been open for 14 days with no activity. Remove stale label or comment or update or this will be closed in 7 days. |
folivoramanh
left a comment
There was a problem hiding this comment.
make sure both pytest and sparrowhawk test pass, and fix the problem of blank space
9d8b75f to
b224888
Compare
|
Thanks for the review. I’ve addressed the comments on this PR.
pytest for Telugu cardinals is passing. Sparrowhawk tests are still pending; I’ll update this PR once they are run. |
folivoramanh
left a comment
There was a problem hiding this comment.
- add init file in data/ (currently only in data/numbers/)
|
Thanks for the follow-up review. Changes Made:
Testing:
Re: Spanish reusable component: replied in the thread with details. |
3fb9a92 to
415b3cf
Compare
Rebased onto staging/telugu_tn_v1 with TE-only changes for PR NVIDIA#447. Signed-off-by: Haridas Nishitha <nharidas@nvidia.com> Co-authored-by: Cursor <cursoragent@cursor.com>
15c6609 to
1e9a8cb
Compare
|
Thanks for the review. Changes Made:
Testing:
|
Rebased onto staging/telugu_tn_v1 with TE-only changes for PR NVIDIA#447. Signed-off-by: Haridas Nishitha <nharidas@nvidia.com> Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
Use TSV-based digit definitions in graph_utils, restore load_labels in utils, and update Jenkinsfile with correct cache paths and TE CI stages. Signed-off-by: Haridas Nishitha <nharidas@nvidia.com> Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
for more information, see https://pre-commit.ci Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
tokenize_and_classify passed a punctuation argument that WordFst no longer accepts after the staging rebase. Signed-off-by: Haridas Nishitha <nharidas@nvidia.com> Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
for more information, see https://pre-commit.ci Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
77befb5 to
0f44929
Compare
Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
|
@mgrafu Thanks for the review. Changes Made:
Testing:
|
Signed-off-by: Haridas Nishitha <nharidas@nvidia.com>
for more information, see https://pre-commit.ci
|
@mgrafu Thanks for the review. Changes Made:
Testing:
|
for more information, see https://pre-commit.ci
|
@mgrafu Thanks for the review. Changes Made:
Testing:
|
| ౫ చాక్లెట్లు ౯ టాఫీలు~ఐదు చాక్లెట్లు తొమ్మిది టాఫీలు | ||
| 5551 అరటిపండ్లు~ఐదు వేల ఐదు వందల యాభై ఒకటి అరటిపండ్లు | ||
| ౫౫౫౧ అరటిపండ్లు~ఐదు వేల ఐదు వందల యాభై ఒకటి అరటిపండ్లు | ||
| 4 మంది మృతి 18 మందికి గాయాలు~నాలుగు మంది మృతి పధ్ధెనిమిది మందికి గాయాలు |
There was a problem hiding this comment.
it looks like we may have a bug -- can you add these and check what happens?
500000010000000~ఐదు కోట్ల ఒక కోటి
5000000010000000~యాభై కోట్ల ఒక కోటి
50000000010000000~ఐదు వందల కోట్ల ఒక కోటి
500000000010000000~ఐదు వేల కోట్ల ఒక కోటి
5000000000010000000~యాభై వేల కోట్ల ఒక కోటి
500000015724739~ఐదు కోట్ల ఒక కోటి యాభై ఏడు లక్షల ఇరవై నాలుగు వేల ఏడు వందల ముప్పై తొమ్మిది
50000000015724739~ఐదు వందల కోట్ల ఒక కోటి యాభై ఏడు లక్షల ఇరవై నాలుగు వేల ఏడు వందల ముప్పై తొమ్మిది
268828015724739~రెండు కోట్ల అరవై ఎనిమిది లక్షల ఎనభై రెండు వేల ఎనిమిది వందల ఒక కోటి యాభై ఏడు లక్షల ఇరవై నాలుగు వేల ఏడు వందల ముప్పై తొమ్మిది
558073012157368~ఐదు కోట్ల యాభై ఎనిమిది లక్షల ఏడు వేల మూడు వందల ఒక కోటి ఇరవై ఒక లక్షల యాభై ఏడు వేల మూడు వందల అరవై ఎనిమిది
6690674013350484~అరవై ఆరు కోట్ల తొంభై లక్షల అరవై ఏడు వేల నాలుగు వందల ఒక కోటి ముప్పై మూడు లక్షల యాభై వేల నాలుగు వందల ఎనభై నాలుగు
500000020000000~ఐదు కోట్ల రెండు కోట్లు
500000110000000~ఐదు కోట్ల పదకొండు కోట్లు
268828025724739~రెండు కోట్ల అరవై ఎనిమిది లక్షల ఎనభై రెండు వేల ఎనిమిది వందల రెండు కోట్ల యాభై ఏడు లక్షల ఇరవై నాలుగు వేల ఏడు వందల ముప్పై తొమ్మిది
What does this PR do ?
Add a one line overview of what this PR aims to accomplish.
Before your PR is "Ready for review"
Pre checks:
git commit -sto sign.pytestor (if your machine does not have GPU)pytest --cpufrom the root folder (given you marked your test cases accordingly@pytest.mark.run_only_on('CPU')).bash tools/text_processing_deployment/export_grammars.sh --MODE=test ...pytestand Sparrowhawk here.__init__.pyfor every folder and subfolder, includingdatafolder which has .TSV files?Copyright (c) 2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved.to all newly added Python files?Copyright 2015 and onwards Google, Inc.. See an example here.try import: ... except: ...) if not already done.PR Type:
If you haven't finished some of the above items you can still open "Draft" PR.