nipype / pydra

Pydra Dataflow Engine
https://nipype.github.io/pydra/
Other
119 stars 57 forks source link

Check DaskWorker performance for all tests #708

Open adi611 opened 9 months ago

adi611 commented 9 months ago

Types of changes

Summary

Check DaskWorker performance for all tests

Checklist

codecov[bot] commented 9 months ago

Codecov Report

Patch coverage: 86.20% and project coverage change: -4.19% :warning:

Comparison is base (0245cdc) 83.42% compared to head (c4691c5) 79.23%. Report is 46 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #708 +/- ## ========================================== - Coverage 83.42% 79.23% -4.19% ========================================== Files 22 23 +1 Lines 4894 4971 +77 Branches 1410 1424 +14 ========================================== - Hits 4083 3939 -144 - Misses 807 827 +20 - Partials 4 205 +201 ``` | [Flag](https://app.codecov.io/gh/nipype/pydra/pull/708/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/nipype/pydra/pull/708/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype) | `79.07% <83.90%> (-4.36%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files Changed](https://app.codecov.io/gh/nipype/pydra/pull/708?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype) | Coverage Δ | | |---|---|---| | [pydra/conftest.py](https://app.codecov.io/gh/nipype/pydra/pull/708?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype#diff-cHlkcmEvY29uZnRlc3QucHk=) | `53.12% <66.66%> (-24.66%)` | :arrow_down: | | [pydra/engine/run\_pickled.py](https://app.codecov.io/gh/nipype/pydra/pull/708?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype#diff-cHlkcmEvZW5naW5lL3J1bl9waWNrbGVkLnB5) | `76.19% <76.19%> (ø)` | | | [pydra/engine/workers.py](https://app.codecov.io/gh/nipype/pydra/pull/708?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype#diff-cHlkcmEvZW5naW5lL3dvcmtlcnMucHk=) | `30.46% <91.66%> (+7.73%)` | :arrow_up: | ... and [15 files with indirect coverage changes](https://app.codecov.io/gh/nipype/pydra/pull/708/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nipype)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

djarecka commented 9 months ago

@adi611 - could you please resolve comflict in the conftest.py