QPropertyAnimation+QGraphicsItem : use 25% of the CPU

[wysota]

We still have no idea how the boundingRect() for your item is defined. What do you set it to?

[pl01]

I have not redefined the 'boundingRect' method !

[wysota]

I know but you are calling setRect() somewhere which defines the bounding rect. By the way, why are you manipulating the animation in the function related to drawing your item?

[pl01]

Yes,

I call : setRect(0, 0, 90, 90); in the "initializeNode()" method.

By the way, why are you manipulating the animation in the function related to drawing your item?

Because, when the item is selected I want to animate a "circle" around the item... a simple color transition... that will highlight the selected item.

[wysota]

I call : setRect(0, 0, 90, 90); in the "initializeNode()" method.

Try enabling cache for your items. However this will surely break your already broken animation code. I wouldn't expect miracles performance wise if you are changing the highlight color constantly. Maybe making the highlight a separate item would help more.

Because, when the item is selected I want to animate a "circle" around the item... a simple color transition... that will highlight the selected item.

paint() routine is for painting, not for starting animations. Manipulate the animation when the selection value changes. And set the loop count to infinite instead of manually checking the state of the animation every frame and restarting it when it finishes its course.

[pl01]

Thanks,

1) I used setCacheMode(QGraphicsItem:eviceCoordinateCache); for caching... Notice I also need anti-aliasing !
2) Sure, I can start the animation somewhere else... but it would'nt change anything to performance ! (I have just try of course) ;-)

[wysota]

1) I used setCacheMode(QGraphicsItem:eviceCoordinateCache); for caching... Notice I also need anti-aliasing !

I don't see how anti-aliasing is relevant here. I would rather use ItemCoordinateCache.

2) Sure, I can start the animation somewhere else... but it would'nt change anything to performance ! (I have just try of course) ;-)

If you change the item every frame, then caching only makes your app slower since it has to be regenerated again and again. If you move the animation outside your item then at least the item itself can be cached properly and will not execute your quite complex code on each frame of the animation. All this does not change the fact that putting animation controlling code into a paint routine is simply wrong.