Closed mgrafu closed 2 months ago
@tbartley94 can you take a quick look at this too?
@mgrafu where'd you move the get_quantity
method?
@tbartley94 removed it altogether -- it's not used anywhere in the code
@tbartley94 removed it altogether -- it's not used anywhere in the code
ah, weird
What does this PR do ?
Address issue #162 -- function with bug is not used in the code so we can just remove it
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.