-
Notifications
You must be signed in to change notification settings - Fork 133
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
Enable outputs to access all inputs #936
Comments
Should the values appear in the viz? Or that isn't part of the requirements? |
If we want to support this we can just add this to the final result computation (E.G. if it's included in the output we just wire it from the input. Seems decidedly reasonable. That said, what's the use-case? This should work if these are inputs are referenced, but why not just join later if they're not? Or is it to leverage the result builder? |
This enables one to pass in inputs and request them as outputs independent of the graph. The use case here is that you want to join some data at the end that is extra and not in the DAG. E.g. extra pandas data.
This enables one to pass in inputs and request them as outputs independent of the graph. The use case here is that you want to join some data at the end that is extra and not in the DAG. E.g. extra pandas data. Adds test cases to check for different angles.
* Enables inputs to be also outputs; Fixes #936 This enables one to pass in inputs and request them as outputs independent of the graph. The use case here is that you want to join some data at the end that is extra and not in the DAG. E.g. extra pandas data. Adds test cases to check for different angles. * Adds two extra checks * PR review feedback
Is your feature request related to a problem? Please describe.
You cannot currently request inputs that are not referenced in the DAG.
Describe the solution you'd like
Allow one to pass in arbitrary inputs not used in the DAG and then be able to request them as outputs.
To recreate
Describe alternatives you've considered
You write a dummy function to contain it as input.
The text was updated successfully, but these errors were encountered: