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

Fix multithreading issues with settings management #2276

Merged
merged 31 commits into from
Jun 20, 2024
Merged

Conversation

Andrew-S-Rosen
Copy link
Member

Closes #2275.

Copy link

codecov bot commented Jun 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.03%. Comparing base (00e0d80) to head (af2db68).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2276      +/-   ##
==========================================
+ Coverage   99.02%   99.03%   +0.01%     
==========================================
  Files          82       82              
  Lines        3387     3431      +44     
==========================================
+ Hits         3354     3398      +44     
  Misses         33       33              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Andrew-S-Rosen Andrew-S-Rosen changed the title [WIP] Fix multithreading issues with settings management Fix multithreading issues with settings management Jun 20, 2024
@Andrew-S-Rosen Andrew-S-Rosen merged commit e3703f8 into main Jun 20, 2024
20 checks passed
@Andrew-S-Rosen Andrew-S-Rosen deleted the concurrency_fix branch June 20, 2024 03:22
Andrew-S-Rosen added a commit that referenced this pull request Jun 20, 2024
…low engine (v2) (#2277)

This PR is a collaboration with @honghuikim. It is a continuation of
#2163 that addresses changes introduced in #2276. For additional history
on this PR, please refer to #2163.

The focus of this PR is to enable dynamic changing of Pydantic model
settings when using an active workflow engine. This is achieved, in
large part, through a new `settings_swap` keyword argument to the
decorator.

Closes #2147.

---------

Co-authored-by: Honghui Kim <itab1107@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Honghui Kim <165984476+honghuikim@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Check if with change_settings() changes global settings on concurrent threads
1 participant