Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[WIP] adapting to 1.0alpha syntax #39

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

[WIP] adapting to 1.0alpha syntax #39

wants to merge 9 commits into from

Conversation

tclose
Copy link

@tclose tclose commented Apr 4, 2025

Adapt the Nipype1 tasks package for 1.0alpha syntax. Converts it from a tasks package to a "pydra.compose" package.

The idea is to be able to do something like

from pydra.compose import nipype1
from nipype.interfaces.some_toolkit import SomeNipypeInterface

SomeNipypeTask = nipype1.define(SomeNipypeInterface())
some_task = SomeNipypeTask(param1=..., param2=...)

outptus = some_task(cache_dir=tmp_path)

@effigies effigies force-pushed the pydra-1.0alpha-syntax branch from 6698956 to 31fc8e6 Compare April 4, 2025 19:52
@effigies effigies force-pushed the pydra-1.0alpha-syntax branch from 31fc8e6 to 7f72f7f Compare April 4, 2025 19:52
@effigies effigies force-pushed the pydra-1.0alpha-syntax branch from 7930e1f to f1ce8d0 Compare April 4, 2025 20:04
@codecov-commenter
Copy link

codecov-commenter commented Apr 4, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 71 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (4c08fab) to head (f5a775e).

Files with missing lines Patch % Lines
pydra/compose/nipype1/builder.py 0.00% 56 Missing ⚠️
pydra/compose/nipype1/tests/test_nipype1task.py 0.00% 12 Missing ⚠️
pydra/compose/nipype1/__init__.py 0.00% 3 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

❗ There is a different number of reports uploaded between BASE (4c08fab) and HEAD (f5a775e). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (4c08fab) HEAD (f5a775e)
5 2
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #39       +/-   ##
==========================================
- Coverage   95.31%   0.00%   -95.32%     
==========================================
  Files           4       4               
  Lines          64      99       +35     
  Branches        7       0        -7     
==========================================
- Hits           61       0       -61     
- Misses          2      99       +97     
+ Partials        1       0        -1

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tclose @codecov-commenter @effigies