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

[DOC] Add doc on joblib + joblib_htcondor #278

Merged
merged 10 commits into from
Oct 23, 2024
Merged

[DOC] Add doc on joblib + joblib_htcondor #278

merged 10 commits into from
Oct 23, 2024

Conversation

fraimondo
Copy link
Contributor

Include documentation on how to use joblib-htcondir with julearn.

Copy link

codecov bot commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.93%. Comparing base (6393826) to head (95928c2).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #278   +/-   ##
=======================================
  Coverage   89.93%   89.93%           
=======================================
  Files          56       56           
  Lines        2494     2494           
  Branches      443      443           
=======================================
  Hits         2243     2243           
  Misses        165      165           
  Partials       86       86           
Flag Coverage Δ
docs 100.00% <ø> (ø)

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

Files with missing lines Coverage Δ
docs/conf.py 100.00% <ø> (ø)

@fraimondo fraimondo marked this pull request as ready for review October 22, 2024 12:45
Copy link

github-actions bot commented Oct 22, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-10-23 10:44 UTC

@fraimondo fraimondo requested a review from synchon October 22, 2024 14:14
Copy link
Member

@synchon synchon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reads very well and quite informative, some general comments before good to go! Also, will need a newsfragment.

docs/conf.py Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to get rid of the commented lines or will we use it later?

docs/selected_deeper_topics/joblib.rst Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
docs/selected_deeper_topics/joblib.rst Outdated Show resolved Hide resolved
@fraimondo fraimondo requested a review from synchon October 23, 2024 08:34
@fraimondo fraimondo merged commit 8793197 into main Oct 23, 2024
12 checks passed
@fraimondo fraimondo deleted the doc/joblib branch October 23, 2024 10:37
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