radical-cybertools / radical.pilot

RADICAL-Pilot
http://radical-cybertools.github.io/radical-pilot/index.html
Other
54 stars 23 forks source link

Google style docstrings. #2911

Closed eirrgang closed 1 year ago

eirrgang commented 1 year ago

Apply PEPs 257, 287.

Fixes #2447

eirrgang commented 1 year ago

There are a number of broken internal and intersphinx links, but I think those can be addressed in follow-ups. I tried to minimize the amount of content updates, but I couldn't resist fixing a number of typos and minor errors, and some executive decisions were made for more compliant docstrings.

codecov[bot] commented 1 year ago

Codecov Report

Merging #2911 (db2c883) into devel (a0a5e3b) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            devel    #2911   +/-   ##
=======================================
  Coverage   40.95%   40.95%           
=======================================
  Files          94       94           
  Lines       10275    10275           
=======================================
  Hits         4208     4208           
  Misses       6067     6067           
Impacted Files Coverage Δ
src/radical/pilot/context.py 50.00% <ø> (-4.55%) :arrow_down:
src/radical/pilot/pilot_description.py 81.57% <ø> (ø)
src/radical/pilot/pilot_manager.py 12.40% <ø> (ø)
src/radical/pilot/pytask.py 63.33% <ø> (ø)
src/radical/pilot/session.py 42.00% <ø> (ø)
src/radical/pilot/staging_directives.py 43.75% <ø> (ø)
src/radical/pilot/states.py 58.59% <ø> (ø)
src/radical/pilot/task.py 71.03% <ø> (ø)
src/radical/pilot/task_description.py 81.41% <ø> (ø)
src/radical/pilot/task_manager.py 21.38% <ø> (ø)
... and 5 more

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

eirrgang commented 1 year ago

If merged before #2559, supersedes #2910 (cherry-picked here).