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 revert MemorizedFunc.call API change #1576

Merged
merged 5 commits into from May 2, 2024

Conversation

tomMoral
Copy link
Contributor

No description provided.

Copy link

codecov bot commented May 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.22%. Comparing base (4db39eb) to head (b0ef6e5).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1576      +/-   ##
==========================================
+ Coverage   95.16%   95.22%   +0.05%     
==========================================
  Files          45       45              
  Lines        7674     7705      +31     
==========================================
+ Hits         7303     7337      +34     
+ Misses        371      368       -3     

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

CHANGES.rst Outdated Show resolved Hide resolved
CHANGES.rst Outdated Show resolved Hide resolved
conftest.py Show resolved Hide resolved
@tomMoral tomMoral merged commit 398d8ee into joblib:main May 2, 2024
19 checks passed
@tomMoral tomMoral deleted the FIX_memorizedfunc_api_change branch May 2, 2024 08:05
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request May 11, 2024
https://build.opensuse.org/request/show/1172881
by user dirkmueller + dimstar_suse
- update to 1.4.2:
  * Due to maintenance issues, 1.4.1 was not valid and we bumped
    the version to 1.4.2
  * Fix a backward incompatible change in MemorizedFunc.call
    which needs to return the metadata. Also make sure that
    NotMemorizedFunc.call return an empty dict for metadata for
    consistency. joblib/joblib#1576
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request May 11, 2024
Release 1.4.2 -- 2024/05/02

Due to maintenance issues, 1.4.1 was not valid and we bumped the version to 1.4.2

- Fix a backward incompatible change in ``MemorizedFunc.call`` which needs to
  return the metadata. Also make sure that ``NotMemorizedFunc.call`` return
  an empty dict for metadata for consistency.
  joblib/joblib#1576
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.

None yet

1 participant