Skip to content

Conversation

vbrodsky
Copy link
Contributor

@vbrodsky vbrodsky commented Sep 20, 2023

Story: https://labelbox.atlassian.net/browse/SDK-35

Update mea upload tests to use valid annotations and add finer-grain error checking

@vbrodsky vbrodsky requested review from a team, apollonin and kkim-labelbox September 20, 2023 23:10
@vbrodsky vbrodsky force-pushed the VB/improve-mea-test_SDK-35 branch from 8d90339 to 23813c7 Compare September 20, 2023 23:13
# TODO: Use a real URI
'instanceURI': "sampleuri",
'colorRGB': [0, 0, 0]
"instanceURI":
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

original mask was invalid for an image we use

@@ -1016,6 +1015,17 @@ def object_predictions(polygon_inference, rectangle_inference, line_inference,
]


@pytest.fixture
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

entity_inference not valid for an image

@@ -11,22 +11,11 @@
"""


def test_create_from_url(model_run_with_data_rows,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicate test, see below

@vbrodsky vbrodsky force-pushed the VB/improve-mea-test_SDK-35 branch from 23813c7 to 5b3110a Compare September 21, 2023 18:23
@vbrodsky vbrodsky force-pushed the VB/improve-mea-test_SDK-35 branch from 5b3110a to 5049882 Compare September 22, 2023 17:20
@vbrodsky vbrodsky force-pushed the VB/improve-mea-test_SDK-35 branch from 3b8796d to a66561a Compare October 2, 2023 20:36
@vbrodsky vbrodsky force-pushed the VB/improve-mea-test_SDK-35 branch from a66561a to 8c3f1b3 Compare October 2, 2023 20:41
@vbrodsky vbrodsky merged commit 40f4dbc into develop Oct 2, 2023
@vbrodsky vbrodsky deleted the VB/improve-mea-test_SDK-35 branch October 2, 2023 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants