Skip to content

Change loop limit from 20 to dec_maxlen - #1464

Merged
wannaphong merged 1 commit into
mainfrom
fix-1462
Jul 29, 2026
Merged

Change loop limit from 20 to dec_maxlen#1464
wannaphong merged 1 commit into
mainfrom
fix-1462

Conversation

@wannaphong

@wannaphong wannaphong commented Jul 29, 2026

Copy link
Copy Markdown
Member

What do these changes do

Fixed #1462 bug: pronunciate(engine="w2p") truncates its output at 20 characters

Your checklist for this pull request

  • Passed code styles and structures
  • Passed code linting checks and unit test

@sonarqubecloud

sonarqubecloud Bot commented Jul 29, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@wannaphong wannaphong mentioned this pull request Jul 29, 2026
Open
@wannaphong
wannaphong merged commit a17dd90 into main Jul 29, 2026
5 checks passed
@wannaphong
wannaphong deleted the fix-1462 branch July 29, 2026 17:06
wannaphong added a commit that referenced this pull request Jul 29, 2026
@wannaphong
Patch #1464
f1b633e 
Replace fixed loop count with dynamic max length
wannaphong added a commit that referenced this pull request Jul 29, 2026
@wannaphong
PyThaiNLP v5.3.5
8675a58 
### Fixed

- Change loop limit from 20 to dec_maxlen #1464
- fix: remove duplicate .replace mid-tone in _clean_ipa #1409
- Fix: CodeQL ReDoS vulnerability in ULMFiT replace_url #1400
- fix: remove unconditional overwrite that disables teacher forcing in Seq2Seq #1380
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bug: pronunciate(engine="w2p") truncates its output at 20 characters

1 participant

@wannaphong