Shuhei Iitsuka comments

Results 11 comments of


                                            Shuhei Iitsuka

Consider to use cElementTree

Or maybe just `from lxml import etree as ET` is sufficient? https://lxml.de/1.3/tutorial.html

ICU-22666 Update ML model to improve Japanese phrase breaking quality

@FrankYFTang Sorry, it seems I run a wrong command and mistakenly closed this PR, which I cannot reopen anymore. I made another PR at https://github.com/unicode-org/icu/pull/2861 with the same change. I...

Clarification on the unicode used on the keys

Hi @Cloudef, Thanks for your insightful feedback! The keys are meant to be a list of codepoints, rather than grapheme clusters. ## Normalization It's a good point. Definitely it would...

fix: Change HEY_LLM's default location from asia-northeast1 to us-central1

@code-review-assist review

fix: Change HEY_LLM's default location from asia-northeast1 to us-central1

@code-review-assist review

dart support

Thank you for your comment. You're correct, we don't currently have an official runtime for Dart. It appears that some individuals have already developed Dart ports for BudouX. Notably, budoux_dart...

ZWSP / WBR insertion causes unintended space trimming on line breaks

This is the rendering of the test case in each browser. cc @kojiishi ![budoux_render](https://github.com/user-attachments/assets/ea570a10-7cf0-4121-8a48-1caf3b34f057)

Numbers become <a href="tel:">links even if <meta name="format-detection" content="telephone=no"> is set on iOS

Thanks for your report. I believe that's because `` outputs the translated HTML to its Shadow DOM, which is isolated from the global document. If that's the case, it will...

Numbers become <a href="tel:">links even if <meta name="format-detection" content="telephone=no"> is set on iOS

Thanks for investigating the issue. Yes, it seems like the root causation. I will keep this issue open for future reference and to track their advances in fixing the issue....

cleanup(deps): Consideration of removing an unnecessary dependency (abort-controller)

+1 to remove `abort-controller`. This issue broke my SvelteKit / AppEngine / Node.js 22 setup. @ddelgrosso1 Do you have a plan to move the minimum supported Node.js version to v16...