Skip to content

Conversation

@benjats07
Copy link
Contributor

The multithreading problem doesn't appears to happen on M1 chip (as it does in x64 processors)
This PR just removes Pool from DocumentLayout.get_elements_from_layout.

@benjats07 benjats07 requested review from qued and removed request for qued March 2, 2023 23:13
@benjats07 benjats07 marked this pull request as draft March 2, 2023 23:28
@benjats07 benjats07 marked this pull request as ready for review March 2, 2023 23:28
@ajjimeno
Copy link
Contributor

ajjimeno commented Mar 3, 2023

It restarted the checks for 5460b58 but there is one more commit for which I cannot start the CI tests. Probably the last commit needs to be ok to go ahead.

@benjats07 benjats07 enabled auto-merge (squash) March 3, 2023 02:12
Copy link
Contributor

@ajjimeno ajjimeno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@benjats07 benjats07 merged commit d332b65 into main Mar 3, 2023
@benjats07 benjats07 deleted the fix/multithreading branch March 3, 2023 02:48
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

Successfully merging this pull request may close these issues.

3 participants