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

niworkflows.utils.bids.collect_data failing to retrieve BOLD images #293

Closed
oesteban opened this issue Jan 28, 2019 · 5 comments
Closed
Labels
bug impact:high Estimated high impact task

Comments

@oesteban
Copy link
Member

It seems that the original bug causing nipreps/fmriprep#1481 is still there (https://neurostars.org/t/fmriprep-from-singularity-unboundlocalerror/3299/9?u=oesteban).

Not getting a 'bold' entry after the BIDSLayout query would explain the original UnboundLocalError symptom. @emdupre first noticed this during #285.

@effigies, any quick thoughts about what could've changed in pybids land that we did not account for in our end?

@oesteban oesteban added the bug label Jan 28, 2019
@effigies
Copy link
Member

We're still depending on pybids 0.6.5... Nothing has changed in that. Unless there's some reason that 0.7.0 is getting pulled in?

@oesteban
Copy link
Member Author

@emdupre could not replicate the original bug, and the error seems persistent only with singularity containers. Maybe a system-wide installation pybids installation is coming into play.

@effigies
Copy link
Member

That's a thought I had, but the paths in their tracebacks are in the same directory as fmriprep.

@effigies
Copy link
Member

Perhaps their singularity image is at fault. Do we have one we can verify works on our data, and give to them to see if they still reproduce the failure?

@effigies effigies added this to To do in pipelines Mar 6, 2019
@oesteban oesteban moved this from To do to Watching in pipelines Mar 6, 2019
@oesteban oesteban added the impact:high Estimated high impact task label Mar 12, 2019
@oesteban
Copy link
Member Author

Seems like the original UnboundLocalError is not happening anymore. Please reopen in case of regression.

pipelines automation moved this from Awaiting for user input to Done Mar 14, 2019
@oesteban oesteban removed this from Done in pipelines Feb 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug impact:high Estimated high impact task
Projects
None yet
Development

No branches or pull requests

2 participants