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

Flaky test for hybrid.Log to shared queue #279

Closed
randomir opened this issue Nov 25, 2022 · 1 comment · Fixed by #286
Closed

Flaky test for hybrid.Log to shared queue #279

randomir opened this issue Nov 25, 2022 · 1 comment · Fixed by #286

Comments

@randomir
Copy link
Member

Example here. But happens on windows as well.

test_shared_memo_queue sometimes fails for memo=multiprocessing.Queue().

Multiprocessing queue probably needs some time for eventual consistency (and we are not giving any).

@randomir
Copy link
Member Author

Reason: see note under Pipes and Queues section:

When an object is put on a queue, the object is pickled and a background thread later flushes the pickled data to an underlying pipe.
[...]
After putting an object on an empty queue there may be an infinitesimal delay before the queue’s empty() method returns False and get_nowait() can return without raising queue.Empty.

randomir added a commit to randomir/dwave-hybrid that referenced this issue Dec 23, 2022
Reason: in mp, put is async, it takes nonzero time.

Fix dwavesystems#279.
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 a pull request may close this issue.

1 participant