Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bashlib: fill ocrd.process.profile logger #1013

Open
bertsky opened this issue Mar 17, 2023 · 1 comment
Open

bashlib: fill ocrd.process.profile logger #1013

bertsky opened this issue Mar 17, 2023 · 1 comment

Comments

@bertsky
Copy link
Collaborator

bertsky commented Mar 17, 2023

We are currently blind in our process/profiling logger to any bashlib-based processors (which step ran and how much time was spent).

Implementing this with bash is probably not so easy – you have to keep track of and aggregate CPU-time from all subprocesses forked by the shell through the script.

@bertsky
Copy link
Collaborator Author

bertsky commented Mar 20, 2023

IMO this should be done based on the recipe described for the API changes.

So this would require a rewrite of all bashlib processors – but we need to do that anyway. (And the page loop is usually based on our input_files array now, but still clumsy and requires lots of boilerplate.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant