-
Notifications
You must be signed in to change notification settings - Fork 61
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
Wrong Pearse code with suffix -e #77
Comments
thanks again .. please state the exact reproduction steps, and the expected and observed behaviour. |
Setup Words to DO_PEARSE_CODES, e.g. by copying the whole WORDS.MDV file like this:
The initial 01 in this part of the output should be 05 (it's a SUFFIX line), but as it comes from the code mentioned above and not from the |
What are Pearse codes based on? Googling "pearse code" doesn't help. Are they a standard thing (in computational linguistics, etc.), or just used in "Pearse" must come from somewhere (someone's name, probably)... |
I'm guessing Roger Pearse. The classics / linguistics / compsci nexus on this github issues board is occasionally useful :) |
Thanks @Ansa211 , I can now reproduce the problem. What is the basis on which you know the correct Pearse codes? Is the system documented somewhere? |
I don't think it is really documented, but from experimenting with it a little bit, this is what I expect:
I think that for the 06 lines of the second type to appear in the output, one has to have at least one of the following lines in WORD.MOD:
and for the 02 lines, Running |
Please check out commit ac417de which breaks the Pearse Code machinery into its own module |
I've checked it out; the pull request has been updated to reflect these changes. |
bin/words sancte perfide improbe
The Pearse code on the "e SUFFIX" line is for some reason 01, not 05 as expected.
I assume one has to look around
src/words_engine/words_engine-list_package.adb
lines479--485
and725--750
, but I was not able to solve the issue.(It's also not clear to me why this is not handled through ADDONS.LAT.)
The text was updated successfully, but these errors were encountered: