Adam Wong

Results 26 comments of Adam Wong

Generally, anything that accepts a reference path (a JSONPath string that references something in JSON payload passed between states) can use a `Placeholder`. https://aws-step-functions-data-science-sdk.readthedocs.io/en/latest/placeholders.html

Clarified what "support placeholders for Map state" means in the initial issue comments: https://github.com/aws/aws-step-functions-data-science-sdk-python/issues/101#issuecomment-917040833. So far, this PR only addresses the 3rd use case, which isn't actually what the requester...

@ca-nguyen This change doesn't just affect Map state. There's the 3 things I mentioned here: https://github.com/aws/aws-step-functions-data-science-sdk-python/issues/101#issuecomment-917040833 InputPath and OutputPath are allowed in all state types except Fail. Please update the...

The description here is a bit confusing. It's not unique to TrainingStep. The same would apply anywhere there is a merging of properties. Another instance is with [`parameters` in ProcessingStep](https://github.com/aws/aws-step-functions-data-science-sdk-python/blob/main/src/stepfunctions/steps/sagemaker.py#L508)....

Sorry for the delay on this one. Our current priority is a v2 release (https://github.com/aws/aws-step-functions-data-science-sdk-python/issues/108) but this is still on our radar. As @xpaulnim mentioned, you are not blocked from...

This is a feature, not a chore. Adding new functionality should be motivated from the customer's POV, not just to fix the tests. That being said, I see some value...

We aren't actively working on JupyterLab support right now, but we could prioritize that depending on customer interest. So far we had three other customers ask about JupyterLab support. Are...

Steps can accept any string for its name as long as it is unique to the workflow. `StepId` only really exists for the SDK's prebuilt [Pipeline](https://aws-step-functions-data-science-sdk.readthedocs.io/en/stable/pipelines.html) classes, which use the...

Thanks for the PR! For integrations that support all 3 integration patterns, we started to introduce a different way to set this using the [`IntegrationPattern`](https://github.com/aws/aws-step-functions-data-science-sdk-python/blob/main/src/stepfunctions/steps/integration_resources.py#L20-L27) enum instead of separate args...

> Would you like me to change the PR to use that pattern? > Seeing that it would be a breaking change as well, I'd probably have to change the...