Skip to content

fix: build WSD Trie after populating dictionary, not before#1388

Merged
bact merged 1 commit intoPyThaiNLP:devfrom
phoneee:fix/wsd-trie-ordering
Mar 29, 2026
Merged

fix: build WSD Trie after populating dictionary, not before#1388
bact merged 1 commit intoPyThaiNLP:devfrom
phoneee:fix/wsd-trie-ordering

Conversation

@phoneee
Copy link
Copy Markdown
Contributor

@phoneee phoneee commented Mar 29, 2026

What do these changes do

Fix WSD Trie being built from empty dictionary — move construction after the population loop

Fixes #1387

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

@sonarqubecloud
Copy link
Copy Markdown

@bact bact added this to PyThaiNLP Mar 29, 2026
@bact bact added the bug bugs in the library label Mar 29, 2026
Copy link
Copy Markdown
Member

@bact bact left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you.

@bact bact merged commit ae3f849 into PyThaiNLP:dev Mar 29, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug bugs in the library

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bug: WSD custom tokenizer has empty vocabulary — Trie built before dictionary is populated

2 participants