adds diff view using diff_match_path.js

OCR4all / LAREX

A semi-automatic open-source tool for Layout Analysis and Region EXtraction on early printed books.

MIT License

179 stars 33 forks source link

adds diff view using diff_match_path.js #259

Closed chaddy314 closed 3 years ago

chaddy314 commented 3 years ago

adds diff view between prediction and ground truth
uses diff-match-patch library which implements a slightly modified Myer's diff algorithm for fast calculation of diffs
adds toggle to only display textlines where a difference has been found
closes #221

chreul commented 3 years ago

looks great but i don't think that we need a prediction AND a diff view, do we? i think the standard view should be only pred or only GT (if available), as it is now. then, when activating "show prediction" (which should probably be placed somewhere else in the menu), the diff view should be shown by default, maybe with the option to deactivate it.

chaddy314 commented 3 years ago

Are you suggesting something like this? "Show Prediction" now shows view 2(see below), "Show Raw" then switches to prediction without diff.

https://user-images.githubusercontent.com/51399945/122180999-3a5dcf80-ce89-11eb-9b3c-ad1959e0fcf5.mp4

diff view off and prediction off(default)
With diff view on and prediction off
With diff view off and prediction on