-
Notifications
You must be signed in to change notification settings - Fork 372
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
adding an option to get all the configurations results #207
Comments
Hi @oriel351, thanks for reaching out :) I think this would be a useful addition. An input parameter such In the meantime, as a temporary solution, you can find the whole array of results (for all the tested configurations) in the output of the Executor step. |
@alexcasalboni yes that sounds good! it would definitely do the work. |
@oriel351 do you have any experience with AWS Step Functions and Node.js? Would you feel comfortable taking a first stab at this? There's no wrong answer, of course :) If not, I'll be happy to jump on this later this month. |
@alexcasalboni yeah why not :) , I'll could get to it towards the weekend, |
@oriel351 did you manage to find some time to work on this? :) |
This was implemented in #231 and merged 👌 I'm closing this issue :) |
Hey,
Currently, when executing the step function, the output would be the duration and cost of the best configuration.
What about adding additional parameter to the input json, so that in the output there will be an array of objects, each corresponds to different memory configuration execution, and let the user handle the data and use it as input to another process.
Thanks!
The text was updated successfully, but these errors were encountered: