8000 Ensure futures are correctly awaited when executed on thread by philippjfr · Pull Request #4031 · holoviz/panel · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@philippjfr
Copy link
Member

In certain scenarios we attempted to dispatch futures for processing on a thread without a running event loop. We now check if we are on a thread and if so we use Bokeh's add_next_tick_callback to process them on the main thread.

@codecov
Copy link
codecov bot commented Oct 19, 2022

Codecov Report

Merging #4031 (8a8f065) into master (2eaa211) will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #4031      +/-   ##
==========================================
- Coverage   83.64%   83.64%   -0.01%     
==========================================
  Files         223      223              
  Lines       33170    33172       +2     
==========================================
  Hits        27746    27746              
- Misses       5424     5426       +2     
Flag Coverage Δ
ui-tests 37.58% <0.00%> (-0.01%) ⬇️
unitexamples-tests 74.65% <66.66%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
panel/io/document.py 55.90% <66.66%> (-0.08%) ⬇️
panel/io/state.py 69.87% <0.00%> (-0.19%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@philippjfr philippjfr merged commit 5d525ae into master Oct 19, 2022
@philippjfr philippjfr deleted the write_errors_on_thread branch October 19, 2022 11:13
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0