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

text offset in morphemes is not checked? #18

Open
kosloot opened this issue Feb 27, 2020 · 2 comments
Open

text offset in morphemes is not checked? #18

kosloot opened this issue Feb 27, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request low priority

Comments

@kosloot
Copy link
Collaborator

kosloot commented Feb 27, 2020

I added a new variant for Arabic to the examples: arabic.2.2.1.folia.xml with offset information everywhere.

Both folialint and foliavalidator accept this file. NICE!

BUT: when I replace the offset in the first morpheme by some way-off value like 666, it is still accepted bij foliavalidator. This is odd.

folialint states:

> folialint arabic.2.2.1.folia.xml 
arabic.2.2.1.folia.xml failed: Unresolvable text: Text for morpheme(ID=, textclass='current'), has incorrect offset 666
	original msg=Unresolvable text: Reference (ID Xar.p.1.s.1.w.1,class='current') found, but no text match at offset=666 Expected 'اسم' but got ''
@proycon proycon transferred this issue from proycon/folia Mar 8, 2020
@proycon
Copy link
Owner

proycon commented Mar 8, 2020

There are some limits to text consistency checking currently (not sure if this was one of them). But it should be expanded and fixed then.

@proycon proycon added the bug Something isn't working label Mar 8, 2020
@proycon
Copy link
Owner

proycon commented Mar 8, 2020

Probably related to #15 and #8

@proycon proycon added enhancement New feature or request low priority and removed bug Something isn't working labels Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request low priority
Projects
None yet
Development

No branches or pull requests

2 participants