Enhancement and Fixes for Bengali Transliteration. #1263

Merged
uroybd merged 13 commits from master into master 5 years ago
uroybd commented 5 years ago (Migrated from github.com)

Bengali pronunciation rules are sometimes complicated. I've tried to solve some problems with that and added some enhancement to make it more realistic.

Bengali pronunciation rules are sometimes complicated. I've tried to solve some problems with that and added some enhancement to make it more realistic.
Owner

Thanks, but this appears to break some testcases, can you have a look?

nodomain.freeyourgadget.gadgetbridge.service.DeviceCommunicationServiceTestCase > testTransliterationSupport FAILED
    java.lang.StringIndexOutOfBoundsException at DeviceCommunicationServiceTestCase.java:113
nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateArabic FAILED
    java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:41
nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateHebrew FAILED
    java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:31
nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateLithuanian FAILED
    java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:77
nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateCyrillic FAILED
    java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:22
nodomain.freeyourgadget.gadgetbridge.test.CalendarEventTest > testSync FAILED
    junit.framework.AssertionFailedError at CalendarEventTest.java:54
Thanks, but this appears to break some testcases, can you have a look? ``` nodomain.freeyourgadget.gadgetbridge.service.DeviceCommunicationServiceTestCase > testTransliterationSupport FAILED java.lang.StringIndexOutOfBoundsException at DeviceCommunicationServiceTestCase.java:113 nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateArabic FAILED java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:41 nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateHebrew FAILED java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:31 nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateLithuanian FAILED java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:77 nodomain.freeyourgadget.gadgetbridge.test.LanguageUtilsTest > testStringTransliterateCyrillic FAILED java.lang.StringIndexOutOfBoundsException at LanguageUtilsTest.java:22 nodomain.freeyourgadget.gadgetbridge.test.CalendarEventTest > testSync FAILED junit.framework.AssertionFailedError at CalendarEventTest.java:54 ```
uroybd commented 5 years ago (Migrated from github.com)

Sorry that I was late. Was out on a tour. Fixed all the problems.

Sorry that I was late. Was out on a tour. Fixed all the problems.
uroybd commented 5 years ago (Migrated from github.com)

Hey, I think the build failure is not an issue from my branch. Right?

Hey, I think the build failure is not an issue from my branch. Right?
uroybd commented 5 years ago (Migrated from github.com)

It worked after rebasing from upstream!

It worked after rebasing from upstream!
Owner

Thanks a lot, great job! 👍 😃

Thanks a lot, great job! :+1: :smiley:
The pull request has been merged as e66d0a2d10.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Freeyourgadget/Gadgetbridge#1263
Loading…
There is no content yet.