Skip to content

Commit c549b58

Browse files
committed
updates to code and test cases and fix some deprecation
1 parent d153f7d commit c549b58

File tree

4 files changed

+11
-18
lines changed

4 files changed

+11
-18
lines changed

labelbox/data/annotation_types/geometry/mask.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ def draw(self,
9393

9494
canvas = canvas if canvas is not None else np.zeros(tuple(dims),
9595
dtype=np.uint8)
96-
canvas[mask.astype(np.bool)] = color
96+
canvas[mask.astype(bool)] = color
9797
return canvas
9898

9999
def _extract_polygons_from_contours(self, contours: List) -> MultiPolygon:

tests/data/metrics/confusion_matrix/test_confusion_matrix_data_row.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
from pytest_cases import fixture_ref
2-
from pytest_cases import pytest_parametrize_plus, fixture_ref
2+
from pytest_cases import parametrize, fixture_ref
33

44
from labelbox.data.metrics.confusion_matrix.confusion_matrix import confusion_matrix_metric
55

66

7-
@pytest_parametrize_plus("tool_examples", [
7+
@parametrize("tool_examples", [
88
fixture_ref('polygon_pairs'),
99
fixture_ref('rectangle_pairs'),
1010
fixture_ref('mask_pairs'),
@@ -27,10 +27,9 @@ def test_overlapping_objects(tool_examples):
2727
expected), f"{example.predictions},{example.ground_truths}"
2828

2929

30-
@pytest_parametrize_plus(
31-
"tool_examples",
32-
[fixture_ref('checklist_pairs'),
33-
fixture_ref('radio_pairs')])
30+
@parametrize("tool_examples",
31+
[fixture_ref('checklist_pairs'),
32+
fixture_ref('radio_pairs')])
3433
def test_overlapping_classifications(tool_examples):
3534
for example in tool_examples:
3635
score = confusion_matrix_metric(example.ground_truths,

tests/data/metrics/confusion_matrix/test_confusion_matrix_feature.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
from pytest_cases import fixture_ref
2-
from pytest_cases import pytest_parametrize_plus, fixture_ref
2+
from pytest_cases import parametrize, fixture_ref
33

44
from labelbox.data.metrics.confusion_matrix.confusion_matrix import feature_confusion_matrix_metric
55

66

7-
@pytest_parametrize_plus("tool_examples", [
7+
@parametrize("tool_examples", [
88
fixture_ref('polygon_pairs'),
99
fixture_ref('rectangle_pairs'),
1010
fixture_ref('mask_pairs'),
@@ -23,10 +23,9 @@ def test_overlapping_objects(tool_examples):
2323
assert metrics == example.expected, f"{example.predictions},{example.ground_truths}"
2424

2525

26-
@pytest_parametrize_plus(
27-
"tool_examples",
28-
[fixture_ref('checklist_pairs'),
29-
fixture_ref('radio_pairs')])
26+
@parametrize("tool_examples",
27+
[fixture_ref('checklist_pairs'),
28+
fixture_ref('radio_pairs')])
3029
def test_overlapping_classifications(tool_examples):
3130
for example in tool_examples:
3231

tests/data/metrics/iou/data_row/test_data_row_iou.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,20 +42,15 @@ def check_iou_checklist(pair, mask=None):
4242
annotation.value.mask.arr = np.frombuffer(
4343
base64.b64decode(annotation.value.mask.url.encode('utf-8')),
4444
dtype=np.uint8).reshape((32, 32, 3))
45-
# print("drowmiou\n\n", data_row_miou(label, prediction),
46-
# pair.data_row_expected, "\n")
4745
assert math.isclose(data_row_miou(label, prediction),
4846
pair.data_row_expected)
4947
assert math.isclose(
5048
miou_metric(label.annotations, prediction.annotations)[0].value,
5149
pair.data_row_expected)
5250
feature_ious = feature_miou_metric(label.annotations,
5351
prediction.annotations)
54-
b = [feature.value for feature in feature_ious]
55-
# print("fiou\n\n", b, pair.expected, "\n")
5652
mapping = {}
5753
for iou in feature_ious:
58-
print(iou)
5954
if not mapping.get(iou.value, None):
6055
mapping[iou.value] = 0
6156
mapping[iou.value] += 1

0 commit comments

Comments
 (0)