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

Another conversion issue #20

Open
ejrau opened this issue Feb 6, 2018 · 2 comments
Open

Another conversion issue #20

ejrau opened this issue Feb 6, 2018 · 2 comments
Assignees

Comments

@ejrau
Copy link

ejrau commented Feb 6, 2018

I noticed another weird error, that I think has to do with matching on text in #1876. This annotation:

screen shot 2018-02-06 at 4 29 17 pm

resulted in this:

<p> <persname target="psn:0418 psn:0560"><wrong why="not displaying properly, missing space between psns. Also, this is listed as Edith and Isabelle, but I think it should just be Edith (psn:0560). In a letter to Carrie dated July 17th, Cather writes that &quot;The Hambourgs will be in Germany then, for the Mozart Festival, and Edith and I will be with old &lt;wrong why=" psn:0126="" psn:0127="">friends</wrong> at St. Cyr sur Mer, near Marseilles."&amp;nbsp;'&gt;We</persname> start south tonight to spend some time with friends on the Mediterranean coast near <placename target="geo:0120">Marseilles</placename>. Best love to you and <persname target="psn:0195">Jim</persname> and the <persname target="psn:0872psn:0179"><wrong why="Not displaying properly, missing space between psns">children</wrong></persname>, and I hope your garden is lovely. </p>

@ejrau
Copy link
Author

ejrau commented Feb 6, 2018

I think the "needs correction" annotation on the word "friends" (annotation #003189) in the letter matched on the word "friends" on the annotation on the word "We" (annotation #003192) resulting in deleting the first part of the sentence and replacing it with the "why" text for the annotation and moving things around .. sort of.

I also noticed that this error caused the camel case in the following tags to be ignored, similar to what happened in the previous issue I submitted, #19

@ejrau
Copy link
Author

ejrau commented Feb 6, 2018

I duplicated the letter as cat.let1876.error.xml in case I move it out of annotonia before this issue is resolved. Thanks!

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

3 participants