amazon-textract-response-parser icon indicating copy to clipboard operation
amazon-textract-response-parser copied to clipboard

`TGeometry.rotate`: Pass degrees to `TPoint.rotate`

Open jonnyarnold opened this issue 1 year ago • 1 comments

There was a bug where TGeometry.rotate didn't pass the degrees parameter to TPoint.rotate, meaning that calls to TGeometry.rotate (and TDocument.rotate) only ever rotated the polygon by 180 degrees.

Potential fix for #133?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

jonnyarnold avatar Jul 05 '24 15:07 jonnyarnold

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 only look at TPoint.rotate which is probably why this fell through the cracks.

athewsey avatar Jul 28 '24 14:07 athewsey