Alex Thewsey

Results 90 comments of Alex Thewsey

Some observations from testing last/this week: - The updated sample as of #26 (use `base` conda env name, `python3` kernel name, `/root` mount point, `0:0` UID:GID) **does seem to work**...

I think ECR public still needs users to [get a docker CLI auth token](https://docs.aws.amazon.com/AmazonECR/latest/public/public-registries.html#public-registry-auth) right? If so, it might be useful to add this step to the README accompanying each...

For the immediate error, It appears that `_block_id_maps` only [gets initialized](https://github.com/aws-samples/amazon-textract-response-parser/blob/c49cca5053a4b053a6ff171fd0adeb31586d053f/src-python/trp/trp2.py#L463) for the block types that are present in the document, which I believe is a bug because `block_id_map(block_type)` &...

`_block_id_maps` initialization was addressed in the linked PR and now released on [PyPI v1.0.3](https://pypi.org/project/amazon-textract-response-parser/1.0.3/). I appreciate this issue was originally reported quite some time ago - If anybody's able to...

Unless I'm mistaken, this is an issue with the detection within Amazon Textract itself, rather than the processing in the Textractor library right? It looks like the preceding record should've...

Even apart from the permissions themselves, I'm having trouble suppressing the generated CDK-Nag errors - I think because of the use of a Singleton Lambda? For example the below within...

As a reminder to myself or info for whoever comes to tackle this first: Just note that this `cd` is usually required for SM Notebook Instances (where the default terminal...

I'm not deep on the Python version of the library, but from what I understand this may be by design... What's your expected behaviour for missing blocks referenced in the...

Following up on this after diving a bit deeper: `TDocument` provides two alternative methods depending on your desired error handling behaviour: - [find_block_by_id()](https://github.com/aws-samples/amazon-textract-response-parser/blob/c49cca5053a4b053a6ff171fd0adeb31586d053f/src-python/trp/trp2.py#L608) returns `None` when no such block exists...

My interim workaround for this is to derive patch classes from the auto-generated models - using the TODO pattern to fix the serialization behaviour - and return those instead from...