Skip to content
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

Closed
oriel351 opened this issue Jul 13, 2023 · 6 comments
Closed

adding an option to get all the configurations results #207

oriel351 opened this issue Jul 13, 2023 · 6 comments

Comments

@oriel351
Copy link

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!

@alexcasalboni
Copy link
Owner

Hi @oriel351, thanks for reaching out :)

I think this would be a useful addition. An input parameter such includeOutputResults: true|false would do the job. What do you think?

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.

@oriel351
Copy link
Author

@alexcasalboni yes that sounds good! it would definitely do the work.
Thanks :)

@alexcasalboni
Copy link
Owner

@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.

@oriel351
Copy link
Author

@alexcasalboni yeah why not :) , I'll could get to it towards the weekend,

@alexcasalboni
Copy link
Owner

@oriel351 did you manage to find some time to work on this? :)

@alexcasalboni
Copy link
Owner

This was implemented in #231 and merged 👌

I'm closing this issue :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants