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

confusion pairs: what's wrong with 'Fux' and 'schänken'? #739

Closed
janschreiber opened this issue Jul 14, 2017 · 6 comments
Closed

confusion pairs: what's wrong with 'Fux' and 'schänken'? #739

janschreiber opened this issue Jul 14, 2017 · 6 comments
Labels

Comments

@janschreiber
Copy link
Contributor

I just double-checked that the pair x:chs is in de_DE.info, but I don't get the expected suggestions for 'Fux' and 'Büxe'. For 'Wixer', 'Waxtum' and 'waxen' the epected suggestions are at the top of the list. Seems strange.
From my tests it seems pretty obvious that the confusion pairs have a noticeable effect in many cases, e.g. for 'Verschpatüng' I get 'Verspätung', which is pretty cool.

@danielnaber
Copy link
Member

This looks like a bug in morfologik. Maybe because the character in question is at the end of the word. I've added a test and will open an issue at morfologik.

@danielnaber
Copy link
Member

Issue added at morfologik/morfologik-stemming#91

@janschreiber
Copy link
Contributor Author

A similar problem with ä/e which I added today: I get the expected result for 'präkere', but for 'schänken' I still get 'Schänken' as the only result. 'schenken' should be number 2 in the list of suggestions I think, but it isn't there.

@danielnaber
Copy link
Member

"Fux -> Fuchs" is fixed now, "schänken" seems to be yet another issue.

@danielnaber
Copy link
Member

The schänke/schenke issue now has an issue at morfologik/morfologik-stemming#94

@danielnaber danielnaber changed the title [de] confusion pairs: what's wrong with 'Fux'? [de] confusion pairs: what's wrong with 'Fux' and 'schänken'? Aug 4, 2017
@danielnaber danielnaber changed the title [de] confusion pairs: what's wrong with 'Fux' and 'schänken'? confusion pairs: what's wrong with 'Fux' and 'schänken'? Feb 2, 2018
@danielnaber
Copy link
Member

Seems to be fixed thanks to Morfologik update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants