spatialmath-python icon indicating copy to clipboard operation
spatialmath-python copied to clipboard

Published 20 hours ago • verified publisher icon bdaiinstitute

Support derived Animate classes in order to use the same axes.

Open tahsinkose opened this issue 3 years ago • 4 comments

Hello there,

I've been using the MATLAB version of these nice toolboxes and just recently finally had a use-case to migrate to Python. I needed to visualize multiple frames in a single animation view. In order to do that, I have to derive from base.animate.Animate and adjust some of the internal data structures. In order to use the same ax as base.animate.Animate does, however, I had to clone this repo and make this change so that all derived classes receive the exact same treatment with the base class. multiple_frames

tahsinkose avatar Mar 21 '22 14:03 tahsinkose

@jhavl this is a rather simple yet capable improvement that provides trivial extensibility of Animate classes. Could you please give it a quick look and merge? 🙏🏼

Cc. @petercorke

tahsinkose avatar Apr 07 '22 22:04 tahsinkose

Looks great! I'll have a look at the change on the weekend. There's more I'd like to do with animations but this seems like a win. Thx for your interest

petercorke avatar Apr 08 '22 02:04 petercorke

@petercorke friendly ping 🙂

tahsinkose avatar Nov 06 '22 20:11 tahsinkose