8000 Add input provider plugins by imagejan · Pull Request #18 · scijava/batch-processor · GitHub
[go: up one dir, main page]

Skip to content

Add input provider plugins #18

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

Merged
merged 1 commit into from
Aug 10, 2018
Merged

Add input provider plugins #18

merged 1 commit into from
Aug 10, 2018

Conversation

imagejan
Copy link
Member

This change makes batch processing extensible by allowing plugins implementing the BatchInputProvider interface to support custom script parameter inputs, such as net.imagej.Dataset which will be provided in the imagej-plugins-batch project.

Because BatchService now extends HandlerService, I needed to rename the supports() methods more concisely into supportsModule() and supportsItem(), to avoid overriding supports() from HandlerService.
Since we are still pre-1.0.0, we can change the API freely, but bump the version to 0.2.0-SNAPSHOT to reflect this API change.

This change makes batch processing extensible by allowing plugins implementing the BatchInputProvider interface to support custom script parameter inputs, such as net.imagej.Dataset which will be provided in the imagej-plugins-batch project.

Because BatchService now extends HandlerService, I needed to rename the supports() methods more concisely into supportsModule() and supportsItem(), to avoid overriding supports() from HandlerService.
Since we are still pre-1.0.0, we can change the API freely, but bump the version to 0.2.0-SNAPSHOT to reflect this API change.
@imagejan imagejan merged commit 48c5b38 into master Aug 10, 2018
@imagejan imagejan deleted the input-providers branch August 10, 2018 14:36
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.

1 participant
0