implement value range with .. operator for pageId

[kba]

When processing works with a non-trivial amount of pages, or when a long process fails in the middle of a run, it would be useful to not only allow comma-separated page ID values but ranges of page IDs.

This PR implements this by introducing the syntax --page-id <startid>..<endid> which

• extracts the first contiguous match of numbers of both <startid> and <endid>
• counts from start to end number and replaces the number in with the zero-padded current number

E.g. you can do ocrd-processor -g PHYS_0001..PHYS_0003 is equivalent to ocrd-processor -g PHYS_0001,PHYS_0002,PHYS_0003 but shorter.

[bertsky]

Very nice!

(Although I would personally find it more intuitive if 01..05 included 05. Most users will probably not be programmers. And even if they are, they might expect semantics as in shell brace expansion.)

[kba]

Although I would personally find it more intuitive if 01..05 included 05

That was my intention ("Generate a list of strings by incrementing the number part of start until including end.") but I got wrong how range works myself :) fixing that.

[kba]

Should we also support this for --file-id?

[codecov-io]

Codecov Report

Merging #672 (7db6083) into master (2e01120) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #672      +/-   ##
==========================================
+ Coverage   78.73%   78.82%   +0.08%     
==========================================
  Files          55       55              
  Lines        3443     3457      +14     
  Branches      698      702       +4     
==========================================
+ Hits         2711     2725      +14     
  Misses        592      592              
  Partials      140      140              

Impacted Files	Coverage Δ
ocrd_utils/ocrd_utils/__init__.py	100.00% <ø> (ø)
ocrd_models/ocrd_models/ocrd_mets.py	93.21% <100.00%> (+0.15%)	⬆️
ocrd_utils/ocrd_utils/str.py	91.00% <100.00%> (+0.89%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e01120...7db6083. Read the comment docs.

[bertsky]

Should we also support this for --file-id?

Not so sure. That's not part of the processor API, and you can still use the regex syntax...

BTW, the pageId range expression must go to the spec, too.