Skip to content

[ENH]: Set context_func in joblib-hotcondor#303

Open
synchon wants to merge 8 commits intomainfrom
fix/configure_with_joblib
Open

[ENH]: Set context_func in joblib-hotcondor#303
synchon wants to merge 8 commits intomainfrom
fix/configure_with_joblib

Conversation

@synchon
Copy link
Copy Markdown
Member

@synchon synchon commented Mar 30, 2026

This PR allows setting of context_func in joblib-htcondor (HTCondor Joblib backend).

@synchon synchon force-pushed the fix/configure_with_joblib branch from a042abc to bd1bf1e Compare March 30, 2026 09:46
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

❌ Patch coverage is 18.91892% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.02%. Comparing base (ff341ff) to head (5dd1f99).

Files with missing lines Patch % Lines
julearn/config.py 3.70% 26 Missing ⚠️
julearn/api.py 60.00% 3 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #303      +/-   ##
==========================================
- Coverage   89.94%   89.02%   -0.92%     
==========================================
  Files          58       58              
  Lines        2825     2862      +37     
  Branches      498      507       +9     
==========================================
+ Hits         2541     2548       +7     
- Misses        182      211      +29     
- Partials      102      103       +1     
Flag Coverage Δ
julearn 89.02% <18.91%> (-0.92%) ⬇️

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

Files with missing lines Coverage Δ
julearn/api.py 87.30% <60.00%> (-1.47%) ⬇️
julearn/config.py 43.47% <3.70%> (-56.53%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 30, 2026

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://juaml.github.io/julearn/pr-preview/pr-303/

Built to branch gh-pages at 2026-03-31 12:35 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@fraimondo
Copy link
Copy Markdown
Contributor

This will also solve juaml/joblib-htcondor#4

In short, it is impossible for joblib-htcondor to re-set all the loggers. What we can do, from julearn, is to make the context function re-set the julearn config, including the logger.

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.

2 participants