Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow mutiple LSTM data #8

Open
makotokato opened this issue May 17, 2021 · 5 comments
Open

Allow mutiple LSTM data #8

makotokato opened this issue May 17, 2021 · 5 comments

Comments

@makotokato
Copy link
Owner

Actually current LSTM uses Thai data only. I need integrate Burmese data too.

@aethanyc
Copy link
Contributor

It's nice to see you're planning to add the support for Burmese! Do you mind add it after uax14_rs is integrated into icu4x? I've already requested the review for integrating uax14_rs into icu4x, and I hope it can be merged soon without blocking any further development.

@makotokato makotokato changed the title Allow mutile LSTM data Allow mutiple LSTM data May 18, 2021
@aethanyc
Copy link
Contributor

aethanyc commented Jun 1, 2021

FYI. The PR importing uax14_rs into icu4x has been merged.

@makotokato
Copy link
Owner Author

Thanks a lot.

I will update https://github.com/makotokato/uax14_rs/tree/multi-lstm

@aethanyc
Copy link
Contributor

aethanyc commented Jun 1, 2021

I will update https://github.com/makotokato/uax14_rs/tree/multi-lstm

Do you mean porting the branch to icu4x?

@makotokato
Copy link
Owner Author

Yes.

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

No branches or pull requests

2 participants