-
Notifications
You must be signed in to change notification settings - Fork 3
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
Configure() method should fail when the inputspec is not compatible, related with #70 #39
Comments
@gab1one Ok if I remove the in progress label? |
I think you are right @Squareys. Can you double-check and close the issue? However, the exception should be treated gracefully. |
The graceful part is usually already handled in KNIME (it should show the invalid settings dialog), but maybe we mean different things? |
We want default KNIME behaviour: Node is red, but can be configured (or so)? |
Wouldn't default behaviour be not to allow configuring invalid settings? And if no auto-setting for the input column selection for example could be found (new node with no node connected at input set), then it is red? |
Exactly. |
If a generated node is configured, e.g. when connected to another node, the configuration must fail if the inputspec is not compatible.
This could be the case if no column of a required type is available.
The text was updated successfully, but these errors were encountered: