Alex Thewsey

Thanks for proposing! This seems like a sensible & small fix to me... Any chance you could add a unit test? It seems like the current ones in [test_trp2.py](https://github.com/aws-samples/amazon-textract-response-parser/blob/master/src-python/tests/test_trp2.py) only...

As far as I can tell, resolving AZ names->IDs would need to be done at deploy time via a custom resource which could make other things more complicated and still...

Hi and thanks for raising this, I see a couple of challenges with this proposal: - We try to keep TRP dependencies as lean as possible (currently just boto3 and...

Thanks @krrishdholakia - I guess I was leaning more towards "all `user` messages since the last `ai` message", rather than "the most recent `user` message", mainly in case messages with...

Yes, for me & my team I think that'd be the most balanced default... But appreciate it's a bit magical so definitely seems worth calling out the behaviour in the...

For our purposes at the moment I think if "all `user` messages since the last `ai` message" was the default then no additional controls would be required. In case others...

Ah whoops I have found a slight difference in my local copy vs the doc example which seems to be what was causing the error, but believe the behavior is...

OK I've confirmed that this is a bug in the CDK: [`Guardrail.generateCfnPiiEntitiesConfig()`](https://github.com/awslabs/generative-ai-cdk-constructs/blob/8a599c34f1d5cfa6820bc6d6166b315d23ef558d/src/cdk-lib/bedrock/guardrails/guardrails.ts#L788) does not actually pass through any of the optional CFnGuardrail [PiiEntityConfigProperty props](https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_bedrock.CfnGuardrail.PiiEntityConfigProperty.html) - just `type` and `action`. From...

Saw same issue when I tried to: ```python from sagemaker.mlops.workflow.pipeline import Pipeline ``` I had: ``` sagemaker-core 2.1.0 sagemaker-train 1.1.0 sagemaker-serve 1.1.0 sagemaker-mlops 1.1.0 ``` ...which seemed to be the...

In general as far as I'm aware, it's ***expected*** that the `model_fn` will be called multiple times because the default behaviour is for the server to load multiple copies of...