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

DOC : Update plugins page #1885

Merged
merged 1 commit into from
Jan 20, 2016

Conversation

OrkoHunter
Copy link
Contributor

Fix #1881

Updated old interpreted outputs.

@OrkoHunter
Copy link
Contributor Author

Shouldn't the cutoff for coveralls be a little more? It failed for -0.0%

@soupault
Copy link
Member

@OrkoHunter we'd like to ask you to prepare couple of tests for the user guide to increase the coverage. (just kidding)

@soupault soupault added 📄 type: Documentation Updates, fixes and additions to documentation ⏩ type: Enhancement Improve existing features labels Jan 20, 2016
'qt': ['imshow', 'imsave', 'imread', 'imread_collection'],
'simpleitk': ['imread', 'imsave', 'imread_collection'],
'test': ['imsave', 'imshow', 'imread', 'imread_collection'],
'tifffile': ['imread', 'imsave', 'imread_collection']}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have imageio in my installation. Anyways, the full output of the command is kind of redundant for a documentation, so I'd suggest to keep those 5 most common plugins only. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds reasonable. Let's just keep the original ones.

@OrkoHunter
Copy link
Contributor Author

@soupault And now it's hung !

@soupault
Copy link
Member

@OrkoHunter yeah, coveralls.io seems to be under maintenance. The PR does nothing to affect coverage though.
Merging, thanks!

soupault added a commit that referenced this pull request Jan 20, 2016
@soupault soupault merged commit ce4bc59 into scikit-image:master Jan 20, 2016
@OrkoHunter
Copy link
Contributor Author

Cool, thanks!

@OrkoHunter OrkoHunter deleted the find_available_plugins branch January 20, 2016 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⏩ type: Enhancement Improve existing features 📄 type: Documentation Updates, fixes and additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants