Closed mgrafu closed 3 weeks ago
Commenting for clarity on review: cardinals and credit cards are going to overlap here. We can permit a higher weight for full credit card strings xxx xxx xxx xxx
but the abbreviated four digit string xxxx
should only be caught by the non-deterministic graph.
@mgrafu
ASSERT:1:01:01 expected:<one hour one minute and one second> but was:<one hour one minutes and one seconds>
Plurals for time was altered.
What does this PR do ?
changes to en tn for tts
Before your PR is "Ready for review"
Pre checks:
git commit -s
to sign.pytest
or (if your machine does not have GPU)pytest --cpu
from the root folder (given you marked your test cases accordingly@pytest.mark.run_only_on('CPU')
). 2) Sparrowhawk testsbash tools/text_processing_deployment/export_grammars.sh --MODE=test ...
pytest
and Sparrowhawk here.__init__.py
for every folder and subfolder, includingdata
folder which has .TSV files?Copyright (c) 2023, 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.