search

PRImA-Research-Lab / prima-page-converter

Command line tool to convert page layout files to the latest PAGE XML format. It supports all previous versions of the PAGE format as well as ALTO XML, FineReader XML, and HOCR
Apache License 2.0
23 stars 6 forks source link

Error writing target ALTO XML file #16

Closed kba closed 3 years ago

kba commented 4 years ago

https://github.com/PRImA-Research-Lab/prima-page-converter/blob/56f960e77d1db84831e40f1d3ffa05a7c0b68898/src/org/primaresearch/dla/page/converter/PageConverter.java#L245

I'm running into this error and I'm stuck on what could be the problem - are there better diagnostics available if writer.write fails?

kba commented 4 years ago

FILE_0001_OCR-D-OCR-CALAMARI.zip

Here's a failing PAGE-XML sample.

chris1010010 commented 3 years ago

Thanks. Found and fixed the problem. Also the error output should be better now

kba commented 3 years ago

Many thanks for the quick and complete fix!

Can you make another release as well please, so we can update https://github.com/UB-Mannheim/ocr-fileformat and https://github.com/OCR-D/ocrd_fileformat ?

chris1010010 commented 3 years ago

Just done the PageConverter release. Is that what you need?

C.

From: Konstantin Baierer notifications@github.com Sent: 18 September 2020 11:16 To: PRImA-Research-Lab/prima-page-converter prima-page-converter@noreply.github.com Cc: Christian Clausner chris1010010@gmail.com; State change state_change@noreply.github.com Subject: Re: [PRImA-Research-Lab/prima-page-converter] Error writing target ALTO XML file (#16)

Many thanks for the quick and complete fix!

Can you make another release as well please, so we can update https://github.com/UB-Mannheim/ocr-fileformat and https://github.com/OCR-D/ocrd_fileformat ?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/PRImA-Research-Lab/prima-page-converter/issues/16#issuecomment-694783765 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AB4A5RX4U76UDJLGGCJWOH3SGMXOTANCNFSM4RPDC7KA . https://github.com/notifications/beacon/AB4A5RUYZ6E7I5OCA6ZMVTTSGMXOTA5CNFSM4RPDC7KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFFUY6FI.gif

kba commented 3 years ago

It is indeed. Thank you very much!