Skip to content

trying pep8 checks using jenkins#11

Closed
okuchaiev wants to merge 4 commits into
masterfrom
testJenkins
Closed

trying pep8 checks using jenkins#11
okuchaiev wants to merge 4 commits into
masterfrom
testJenkins

Conversation

@okuchaiev
Copy link
Copy Markdown
Collaborator

No description provided.

@okuchaiev okuchaiev closed this Sep 17, 2019
@okuchaiev okuchaiev deleted the testJenkins branch September 17, 2019 17:53
XuesongYang pushed a commit to XuesongYang/NeMo that referenced this pull request May 31, 2024
…honemes (NVIDIA-NeMo#11)

Signed-off-by: Paarth Neekhara <paarth.n@gmail.com>
dcurran90 pushed a commit to dcurran90/NeMo that referenced this pull request Oct 15, 2024
Add cli serve with basic functionality
shivansh023023 pushed a commit to shivansh023023/NeMo that referenced this pull request May 14, 2026
Replaces 15 bare print() calls across three ASR utility files with
the NeMo logger (from nemo.utils import logging) as required by
CONTRIBUTING.md rule NVIDIA-NeMo#11: 'Loggers are preferred to print.'

- vad_utils.py: 3 calls replaced
- streaming_utils.py: 7 calls replaced (stdlib logging import updated)
- transcribe_utils.py: 5 calls replaced

Logging levels assigned based on message severity (info/warning/debug).

Signed-off-by: Shivansh Singh <singhshivansh023@gmail.com>
shivansh023023 pushed a commit to shivansh023023/NeMo that referenced this pull request May 14, 2026
Replaces 15 bare print() calls across three ASR utility files with
the NeMo logger (from nemo.utils import logging) as required by
CONTRIBUTING.md rule NVIDIA-NeMo#11: 'Loggers are preferred to print.'

- vad_utils.py: 3 calls replaced
- streaming_utils.py: 7 calls replaced (stdlib logging import updated)
- transcribe_utils.py: 5 calls replaced

Logging levels assigned based on message severity (info/warning/debug).

Signed-off-by: Shivansh Singh <singhshivansh023@gmail.com>
Signed-off-by: Hackathon User <hackathon@example.com>
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.

1 participant