rerun icon indicating copy to clipboard operation
rerun copied to clipboard

Increase thickness of lines drawn between keypoint connections

Open palol opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe. I have not been able to find an example that demonstrates drawing thicker/thinner keypoint skeletons using the AnnotationContext class. Currently, it seems like the only solution is to instead draw the skeleton using LineStrips.

Describe the solution you'd like Would it be possible to expose another variable like radii to the AnnotationContext class, which would enable adjusting the thickness of the drawn keypoint skeleton.

Describe alternatives you've considered Alternatively, including a code example using LineStrips to draw a custom skeleton for a set of keypoints.

Additional context Example change

rr.log(
        "/",
        rr.AnnotationContext(
            rr.ClassDescription(
                info=rr.AnnotationInfo(id=0, label="Person"),
                keypoint_annotations=[rr.AnnotationInfo(id=lm.value, label=lm.name) for lm in mp_pose.PoseLandmark],
                keypoint_connections=mp_pose.POSE_CONNECTIONS,
                **radii=0.25**
            )
        ),
        timeless=True,
    )

palol avatar Mar 28 '24 19:03 palol

Makes sense.

In the short term you can actually influence this behavior by selecting the space view and then modifying the "Default size" of line radius.

Exa: image

However, I agree we should be able to control this explicitly via the API.

jleibs avatar Mar 28 '24 19:03 jleibs

Makes sense.

In the short term you can actually influence this behavior by selecting the space view and then modifying the "Default size" of line radius.

However, I agree we should be able to control this explicitly via the API.

Thank you! I new there had to be a global setting somewhere in the viewer.

palol avatar Mar 29 '24 00:03 palol

As pointed out in

  • https://github.com/rerun-io/rerun/issues/6064

this should naturally also be per class, not just on the entire context

Wumpf avatar Apr 23 '24 12:04 Wumpf

looks like we regressed further on this: now the workaround doesn't work either anymore

Wumpf avatar Oct 22 '24 09:10 Wumpf