Compose raising when keypoints are configured but no keypoints are passed about albumentations

[ternaus]

Would this do the job?

if keypoints is None:
    result = cursed(image=image, keypoints=[])
    processed_keypoints = None
else:
    result = cursed(image=image, keypoints=keypoints)
    processed_keypoints = result["keypoints"]
    
processed_image = result["image"]

if processed_keypoints is None:
   => self_supervision
else:
  => supervision

[LucaBonfiglioli]

Yep, that's pretty much how I solved it too. But having to manually check each time a Compose is called should not be client code responsibility in my opinion. Also, keep in mind that bounding boxes must be considered in this check as well, further messing the code up.

Wouldn't it be so much better like this?

keys_to_targets: dict[str, Any] = get_stuff_to_transform(...)
processed = compose(**keys_to_targets)

[ternaus]

It is hard to distinguish if person did not pass boxes or keypoints for a reason, like in your case, or just forgot about it.

But we can add functionality and pass keypoints = None in the case when keypoints are missing, but user still wants transform to work.

I think, in such case:

keys_to_targets: dict[str, Any] = get_stuff_to_transform(...)
processed = compose(**keys_to_targets)

will work

[ternaus]

Addressed in #1697