I still don't think this is correct. Should it be _frame.shape.length?

You need some tests that ensure you have the right variable names here. I think on the python side you call the a length, not a height. Why doesn't an error raise if there is no height attribute?

The cylinders in the example are still too long!