dinglehopper

dinglehopper copied to clipboard

Test reports broken

2

- [ ] HTTPError: Response code 400 (Authentication information is not given in the correct format. Check the value of Authorization header.) - [x] Node.js 16 actions are deprecated. Please...

mikegerber

✔ Test on Python 3.13

3

Dependencies aren't ready yet (e.g. `orjson`), so this is a draft until fixed.

mikegerber

Windows support

5

dinglehopper does not seem to run on Windows, because Microsoft Store has Python 3.12 and OCR-D core is not ready yet (https://github.com/OCR-D/core/issues/1212). We already test Python 3.12, why did we...

mikegerber

We would have caught #109 early if we had tests of the CLI.

mikegerber

I am bit confused by the results now, because the test reports are not under "Tests Report" -> Needs review.

mikegerber

Review error rate definitions etc.

2

mikegerber

dinglehopper --version, ocrd-dinglehopper --version

1

Apparently there is no --version parameter and it would be good to have one - now that we have versions. - [ ] Add --version - [ ] Now would...

mikegerber

GitHub Actions

1

- [x] Port CI from CircleCI to GitHub Actions - [x] Run tests in GitHub Actions - [x] Replace CI badge - [x] Review Black job - [x] Port Black...

mikegerber

I decided to merge #83 as is because of my vacation, but I had some minor changes to consider. (List also includes things that I stumbled upon while reviewing, that...

mikegerber

Review notebook/*.ipynb

1

mikegerber