Skip to content

Conversation

kangkengkhadev
Copy link
Contributor

What does this changes

Brief summary of the changes

What was wrong

Description of what was the root cause of the issue.

How this fixes it

Description of how the changes fix the issue.

Fixes #...

Your checklist for this pull request

🚨Please review the guidelines for contributing to this repository.

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

kangkengkhadev and others added 2 commits April 3, 2023 03:31
 for the check_aek_too function, we can now use dead_syllable as aek which is allowed for Thai poem such as klong
@coveralls
Copy link

coveralls commented Apr 2, 2023

Coverage Status

Coverage: 57.014% (-0.1%) from 57.136% when pulling 686bcd0 on kangkengkhadev:dev into b881672 on PyThaiNLP:dev.

Thai tonal word checker
:param str or list[str] text: Thai word or list of Thai words
:param str or list[str] text: Thai word or list of Thai words, bool dead_syllable_as_aek: if True, dead syllable will be considered as aek
Copy link
Member

@wannaphong wannaphong Apr 3, 2023

Choose a reason for hiding this comment

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

Can you change this line by create new line?

:param Union[List[str], str] text: Thai word or list of Thai words
:param bool dead_syllable_as_aek: if True, dead syllable will be considered as aek

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@wannaphong
Copy link
Member

Thank you!

@wannaphong wannaphong merged commit 1dd81c8 into PyThaiNLP:dev Apr 4, 2023
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.

4 participants