Add Inputs class #387
Merged
Add Inputs class #387
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This enables some new use cases: e.g. it becomes possible to make a DynamicCommand anonymous subclass as a local variable in a method, for the purpose of using it once, without registering it in the context.
The idea is to build up the inputs with code dynamically in Java, run the command (which does nothing when executed) to exploit the module preprocessing chain, and then harvest the final input values. In this way, the user can be prompted for inputs dynamically from Java code, roughly similar to the ij.gui.GenericDialog class of ImageJ 1.x.
Co-authored-by: Deborah Schmidt <mail@frauzufall.de>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi @ctrueden,
during the Hackathon last December you made this branch after we talked with @tischi about having something like a generic dialog for asking the user to set input values.
I made the tests work, If you agree with the changes, I would squash the test commits.
Let me know if there is anything else I can do to get this merged.