Skip to content

Add nipype_crash_search script #1422

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

Closed
wants to merge 83 commits into from
Closed

Conversation

alexsavio
Copy link
Contributor

This is a script to search through a log folder for regular expressions in the tracebacks of the .pklz files.

Is it useful?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 72.538% when pulling eea0b9a on alexsavio:crsrch into 4b392d5 on nipy:master.

akeshavan and others added 27 commits April 7, 2016 11:32
* Use nibabel instead of SimpleITK to determine image shape
* Do not convert .mgz to .nii
* Simplify error checking and origvol_names with comprehensions
* PEP8 cleanup in/around checkT1s
fix: segstats avgwf_txt_file can be a bool without error
@satra
Copy link
Member

satra commented Apr 16, 2016

thanks @alexsavio - for now i think this is fine. in the future i'd like to move this to a database search to speed things up.

@satra
Copy link
Member

satra commented Apr 16, 2016

could you please update the CHANGES file and add some documentation (perhaps at the same place as nipype_display_crash)?

@alexsavio
Copy link
Contributor Author

Please let me know if that is ok.

@satra
Copy link
Member

satra commented Apr 18, 2016

@alexsavio - something weird is happening with this merge (there are too many files included that are already in master). can you close this pull request and make a change on top of current master and send?

@alexsavio
Copy link
Contributor Author

I am sorry, yes, I still don't know how to deal with rebase/merge... I will create a new PR.

@alexsavio alexsavio closed this Apr 18, 2016
@alexsavio
Copy link
Contributor Author

Here it is: #1438

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.

7 participants