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

Img proc api #291

Closed
wants to merge 9 commits into from
Closed

Img proc api #291

wants to merge 9 commits into from

Conversation

giltis
Copy link

@giltis giltis commented Jun 24, 2015

Per our conversations about where to place, and how to include the compilations of image processing tools for segmentation, analysis and quantification, this PR adds tools for image filtering, morphological operations, and registration to the API, and (via VTTools) makes them accessible for use in VisTrails for our users.

@tacaswell
Copy link
Member

I have no problems with this. Should add a test file which just does an import foo for all of these. If we want to be really cranky check that their name spaces match the expected all and will fail if we add or remove anything from them. We don't need to do functionality checks as they are done else where (either by us or by their third-party sources).

@ericdill
Copy link
Member

I would argue that this sort of 'helpful namespacing' is actually harmful (re: pylab). If we are going to reduce the barrier to new users, we should be providing awesome examples/demos in python scripts and ipython notebooks! Closing for now.

@ericdill ericdill closed this Jul 21, 2015
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.

None yet

3 participants