AllenInstitute / segmentation-labeling-app

Data pipeline and UI for human labeling of putative ROIs from 2p cell segmentations
Other
0 stars 0 forks source link

Fix post-annotation lambda to properly return data #78

Closed kschelonka closed 4 years ago

kschelonka commented 4 years ago

Compute the majority label (if it exists) when consolidating annotations.

Add CloudWatch to permissions so Lambda can write logs.

Give full s3 access since we aren't using SageMaker prefix in our bucket names.

Update tests with real sample payload and events from Sagemaker.


Two ways of testing the lambda:

sam local invoke PostAnnotationLabelingFunction --event tests/lambdas/resources/consolidation_request.json => Gets real data stored in s3 and processes it Also have coverage with s3 service mocked out using pytest.

Validation: Made a test job to do the full integration Screen Shot 2020-05-06 at 4 28 45 PM Object Summary: https://us-west-2.console.aws.amazon.com/sagemaker/groundtruth?region=us-west-2#/details-object/lambda-test-4