[Panel-devel] Smooth movement of clock hands on plasma applet

[Robert Knight]

> So, I'd say that I accept the patch (if nobody has any further objections),
> while still keeping this behiavour optional, and disabled by default.

I object.  The processor usage this patch causes in its current state
makes it quite obviously out of the question for inclusion in an
important and always-active desktop widget.  It is however an
interesting proof of concept and it started a useful discussion on
conservation of power in plasma and optimizing the clock's rendering.

Seriously though, I have seen this before where an immature patch for
a feature is proposed to which other developers don't agree, but to
avoid causing offense the response is something along the lines of
"include it ... but make it optional and disabled by default".  I
cannot speak for others, but as a user I don't appreciate settings
dialogs cluttered with options for features which don't work properly.

I think a better response is to comment on the basic idea ( in this
case smooth movement of clock hands ) and what needs to be done to
make it suitable for inclusion.  In this case I would suggest that it
would be okay if it could be optimized to reduce processor usage
considerably and with further optimizations ( eg. turning off
completely ) when the system goes into power-conserving mode ( checked
using Solid's PowerManager::appShouldConserveResources() flag ).

Regards,
Robert.

On 30/05/07, Riccardo Iaconelli <ruphy at fsfe.org> wrote:
> Il Sunday 27 May 2007 08:41:20 Aaron J. Seigo ha scritto:
> > to be honest, the default will need to be witout a seconds hand so that we
> > can have the timer fire once per minute. i got it down to this with
> > kicker's clock and, for power reason, we'll need to do the same.
> >
> > that said, i'm fine with this sexiness in there as long as it is optional
> > =)
> >
> > ruphy is managing the development of the clock from here out, though, so
> > i'll wait for his input as well...
> Sorry for the retard, but this is the week with the final test for school....
> and I'm a bit busy... hopefully today will be the last really busy day.
> I agree with Aaron here, I'd make something like kpersonalizer already do
> (slow processor - fast processor), even in 3.x.
> The second hand as well will be made optional, actually I'm not really happy
> with the clock taking 100% of the CPU twice a second.
> For the rest the paint algorithm will need to be optimized for sure....
>
> So, I'd say that I accept the patch (if nobody has any further objections),
> while still keeping this behiavour optional, and disabled by default.
>
> Bye,
> -Riccardo
> --
> GPG key:
> 3D0F6376
> -----
> Pace Peace Paix Paz Frieden Pax Pokój Friður Fred Béke 和平
> Hasiti Lapé Hetep Malu Mир Wolakota Santiphap Irini Peoch
> Shanti Vrede Baris Rój Mír Taika Rongo Sulh Py'guapy 평화
> _______________________________________________
> Panel-devel mailing list
> Panel-devel at kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>