srophe / caesarea-data

Data repository for Caesarea-Maritima.org
0 stars 2 forks source link

Test out why pipe character conversion to <lb/> isn't working consistently #166

Closed wlpotter closed 1 year ago

wlpotter commented 1 year ago

Cf. #150. It seems like it only worked part of the time. Need to dig into this a bit more.

wlpotter commented 1 year ago

Want to test the post-processor fixes a bit more to make sure they work

wlpotter commented 1 year ago

For testing, the records in #170 have 667 instances of a "|" character, so there should be 667 <lb/> after post-processing.

wlpotter commented 1 year ago

This still isn't working. I wonder if it occurs when there are interspersed foreign or title elements? I think I need to revisit this algorithm.

wlpotter commented 1 year ago

Also while we're at it make sure it handles pipe characters in children of the excerpt ab as well (e.g., foreign, title, etc.). 422 is an example with it in a placeName element.

wlpotter commented 1 year ago

This should be fixed.