duckling icon indicating copy to clipboard operation
duckling copied to clipboard

Enhancement + Fixes for Hebrew Language - Duration+TimeGrain+Time+Amount

Open idanadut opened this issue 4 years ago • 4 comments

("idanadut" fork commit)

Handled preposition, hebrew plural forms, grammer miseaks and added support for new cases that are different from the "english language format" that some of the rules had to the correct Hebrew form.

Mostly these enhancement are about new hebrew grammer options to write time intervals, catch simple times correcly, and handle more complex forms. All changes were tested carefully and extensively to verify all cases (and some tests added to corpus) -> examples: Phrases such as "ב15 לחודש" (which mean "on the 15th of this month"), "שנה שעברה" (which means "last year"), "לפני שלושה חודשים" (which means "three months ago"), "חצי שנה אחרונה" ("past half a year"), "בין 15 לנובמבר לבין 18 לדצמבר" ("between 15th of November to 18th of December") - will now work correctly!

idanadut avatar Jan 28 '20 20:01 idanadut

What does it mean that the Travis CI build failed("The command "cabal check" exited with 1.")? One build did succeed and build was fine on local. what changes do I need to make?

idanadut avatar Jan 29 '20 08:01 idanadut

The travis build has a simple fix (https://github.com/facebook/duckling/pull/406/commits/361cf01d8b8e9a56189d76806c26fc077f14d5c6) but it seems PR are not being reviewed/merged in this project anymore. (Edit: the commit is now merged)

ccoreilly avatar Feb 15 '20 10:02 ccoreilly

@idanadut can this be split into separate PRs? one per dimension?

chessai avatar Aug 11 '21 19:08 chessai

@chessai Yes, sure. I'm not sure I'll be able to get to it any time soon , so you are welcome to do so if you'd like.

idanadut avatar Aug 12 '21 08:08 idanadut