Help trying to add support for additional language

[hershkoy]

I am trying to add support for Hebrew, following the instructions given in the Contributing to LUIS with Microsoft/Recognizers-Text.

I forked the repo to my account. Currently working on the Numbers module.
You can see my changes in the commit

I run npm run build to compile the project, and it generated the hebrew compiled folder:

However it doesn't work. I tried running the test, but it just 'skips' the Hebrew one.

So I tried running specifically the Hebrew test using, and I am getting error that there are no tests:

npm test test/runner.js --match='*Hebrew*'

  × Couldn’t find any matching tests

I tried changing JavaScript\samples\simple-console\index.js and switched the defaultCulture:

const defaultCulture = Recognizers.Culture.Hebrew;

But it is not recognizing my Hebrew numbers.

What am I missing?

[Börje Karlsson (tellarin)]

Hi hershkoy, thanks for contributing to the project.

One item that is unclear in that post is that you need to register the new language in the main recognizers class for each type and also with the test running code. Below is a list of related files:

• ./test/cultures.js
• ./recognizers-text/src/culture.ts
• ./recognizers-number/src/number/numberRecognizer.ts

You also need to further edit ./recognizers-number/src/recognizers-text-number.ts
To export { HebrewNumeric } from "./resources/hebrewNumeric";.

Feel free to open a draft PR and we can take a look and add comments in the missing pieces.

[Börje Karlsson (tellarin)]

hershkoy did this help unblock you?

[hershkoy]

Sorry, I had a busy week
I added some more places where I missed adding the Hebrew definitions, but it still doesn't work.
I will try to update soon.
About the pull request: Should I do it in a separate branch or something?

[Börje Karlsson (tellarin)]

In your fork of the repo, create a new branch, then make a PR from there and mark it as Draft. I'll try to take a closer look ASAP.

[hershkoy]

done:
PR: #2699