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

DEV: Added basic img_proc functions to modules.yaml for mrg #40

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

giltis
Copy link
Contributor

@giltis giltis commented Dec 10, 2014

Tool sets added thus far include:
Thresholding
Filtering
Morphology
Transformation

This merge corresponds to the additions made in my NSLS2/science_api PR

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 38a4f1f on giltis:science_api into bfe7524 on Nikea:master.

Gabriel Iltis added 4 commits January 9, 2015 18:54
Tool sets added thus far include:
Thresholding
Filtering
Morphology
Transformation
Removed the ff_image api structure file and reorganized the
image processing tools and functions in accordance with the changes
incorporated into wrap_lib. Image processing tools are now included
according to tool function or category.
@giltis
Copy link
Contributor Author

giltis commented Jan 20, 2015

Finished conversion of the image processing tools api so that it matches the new wrap_lib format. These new changes should be ready for merging as soon as the skxray image processing tools are merged. However, autowrap will break if this is merged before the skxray PR.

@@ -74,6 +88,154 @@ autowrap_func:
- func_name: estimate_d_blind
module_path: skxray.calibration
namespace: calibration
#- func_name: find_ring_center_acorr_iD
Copy link
Member

Choose a reason for hiding this comment

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

Can you delete these? I don't think they are used anywhere.

@tacaswell
Copy link
Member

This needs a rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants