Ok, I just did a quick test, with my contact list of 76 contacts... I had 66 offline contacts, I expand/collapse the offline group, it's instantenuous, the old CL does it at the same speed, now I open my task manager, set the update speed to High, then do it again, the old CL has a peak of 32% of CPU when I expand and 11% of CPU when I collapse.. now on the new CL, I do the same, expanding seems to give 5% of my CPU and collapsing 3%... this means that this is what the redraw takes... Now I noticed something else, resizing the old CL takes maybe 30% of CPU, while resizing the new CL takes 50% of the CPU (it's a dual core, so it's the max, let's say it takes 100% of the cpu...), and it is really visible, when I resize, the old CL takes no time, while the new CL seems to be VERY slow at the resize... now I want to know why, I look at the difference and it seems the new CL redraws itself on every resize event, so it checks all the contacts names crops them if they are too long, replaces the smileys,e tc.. on EVERY resize.. this means a hundred times per second, this is not the case with the old CL, the old CL redraws itself once the user releases the mouse and stops resizing... this has the effect of the nickname truncation being visible either on the fly or at the end of the resize.. This is something that we can take care of, but it also shows us that the new CL CAN be redrawn hundreds times per second, I just can't imagine how the old CL would behavior if we redrew it on every resize event. this basically means that the new CL is ALOT fast than the old one, we could fix the performance issue of the resize quite easily, but overall, it still is an improvement.. in other words, why wait ? :)
p.s.: I had to fix a couple of things before the CL was able to show correctly without bugging... KKRT On Mon, Dec 04, 2006 at 05:53:23AM -0500, Youness Alaoui wrote: > Hi Karel, > Thanks for taking the time to write us. > About clgui.tcl, yes, that's the 3rd one.. the design is good because of the > snit OO thing, but I don't know about bg image or > performance. about the old/new cl that burger started, I don't know either > about the performance, but I think that burger said > it was very fast and afaik, it IS very fast, or at least, no flickering and a > lot faster than the current CL. > I think we should go with that one burger started, and we should first do > some tests with it, by simulating (or using an a huge > account) a CL with 500 users, if it goes well in those conditions, then we're > sure it's fast enough, if it's faster than the > current CL, even if it's slow, it still is an improvement, and if it's slow, > then I think we should be able to fix it the right > way : do a performance test on it to know which functions take the longest! > then optimize those bits instead of a total > rewrite. We can't use that method on the current CL because the code is too > badly written and it really needs a rewrite, as > long as the design is good, we should go with it. And since I think that Phil > has been working on that one, and he said that > the design was good, then I'd say stick with it, optimize it later (or during > the development). > All we want is for it to be faster than the current CL, if it's doable, then > we're good, all we get is a bg image and drag and > drop, it's still an improvement.. if the performance is worst, then there's > the problem. > what do you think? > > KKRT > > On Mon, Dec 04, 2006 at 11:42:16AM +0100, Karel Demeyer wrote: > > I didn't get the other mails (because of my spam filter I think) but I > > see Phil calling me a liar :p > > > > What I said is that the one I made with snit objects (clgui.tcl) doesn't > > support and never will support background images. But, I think this is > > a better design then the guicontactlist.tcl one (the one you want to > > make default now, which was started by Burger but where I did a lot of > > work on). Tom's contactlist.tcl is somewhat the same design as > > clgui.tcl but uses his own widgets for placing widgets on a canvas. > > This makes background images etc possible and is very beautiful, well > > designed etc... but imo it's rather slow. guicontactlist.tcl was > > promising and I believed it wasn't going to be slow but now it just it > > while there's a lot still unfinished. > > I vote (if that's still possible) for something like clgui.tcl but > > without first changing the whole abook.tcl code to something snit-based. > > This is what stopped me further working on it. > > I also said, once you support a background image but then see the new CL > > you have is too slow you can't change easily to one without background > > images. > > > > > > > > Karel. > > > > > > > > Op maandag 04-12-2006 om 00:53 uur [tijdzone -0500], schreef Youness > > Alaoui: > > > wooow.. wait! there are THREE new CLs... lol, > > > the one that burger did with a canvas (that was almost functional), then > > > the one Karel did with snit objects (as a proof of concept), then > > > the one Tom did with a 'very nice ui'... > > > At first, I was talking about the one done by burger with a canvas and on > > > which everyone worked so hard to make it almost complete... > > > if you want to choose another one of those, then go with it, but those > > > are too 'unfinished' to be easable, so it will be a full rewrite of > > > the CL instead of just bugfixing what was left to do... > > > > > > KKRT > > > > > > > > > On Sun, Dec 03, 2006 at 07:52:23PM +0000, [EMAIL PROTECTED] wrote: > > > > On 12/3/06, Philippe Valembois - Phil <[EMAIL PROTECTED]> > > > > wrote: > > > > > > > > > >Hum... so Karel is a liar... he said me we can't add a background on > > > > >your > > > > >new > > > > >CL... > > > > > > > > > > > > Well, it's a canvas, so you can. At the momnent it uses a scalable-bg, > > > > but > > > > it could easily be changed so it's just an image.... whatever. A good > > > > feature of this CL is it has a search box to search for contacts (no > > > > longer > > > > does it as-you-type though, that was too slow!) To add some test > > > > contacts to > > > > the list you can call "speedycontacts" (no quotes obv). Also correction > > > > to > > > > my previous post saying how to show this CL, it should be: > > > > > > > > pack forget .main;source contactlist.tcl;pack [contactlist .cl] -expand > > > > true > > > > -fill both > > > > > > > > > > > > Well... So maybe it's a bad choice I did... > > > > >Why Youness isn't here ? :d > > > > >Well... Which CL will we choose ? > > > > >Phil > > > > > > > > > >Le Sunday 03 December 2006 19:24, [EMAIL PROTECTED] a écrit: > > > > >> On 12/3/06, Philippe Valembois - Phil < > > > > >[EMAIL PROTECTED]> > > > > >> > > > > >> wrote: > > > > >> > We are using the newCL from guicontactlist done originally by > > > > >> > Karel -> > > > > >> > look at > > > > >> > all my commits on it :p > > > > >> > > > > >> Okay! > > > > >> > > > > >> And the contact list in contactlist.tcl can't include a background... > > > > >> > > > > >> > > > > >> It can and does, but the question is academic now we aren't using it > > > > >> :P > > > > >> > > > > >> Phil > > > > >> > > > > >> > Le Sunday 03 December 2006 19:07, [EMAIL PROTECTED] a écrit: > > > > >> > > Hold on a minute everyone, which contact list are we using? > > > > >> > > There's > > > > >two > > > > >> > > "new" ones, the one originally made (by Burger to start with, I > > > > >think) > > > > >> > > which is in guicontactlist.tcl and the one I made in > > > > >> > > contactlist.tcl > > > > >. > > > > >> > > > > > >> > (you > > > > >> > > > > > >> > > can see this by doing pack forget main.tcl;source contactlist.tcl > > > > >;pack > > > > >> > > [contactlist .cl] -expand true -fill both > > > > >> > > > > > > >> > > We need to get rid of whichever we arent using... (I dont mind > > > > >which, > > > > >> > > although I think the more recent one (contactlist.tcl) is less > > > > >cludgy > > > > >> > > > > > >> > and > > > > >> > > > > > >> > > has more potential for graphical niceness...it might be a little > > > > >> > > bit > > > > >> > > > > > >> > slower > > > > >> > > > > > >> > > though... but it doesnt redraw everytime something happens (dont > > > > >know > > > > >> > > if other one does...) be aware it also isnt hooked up to > > > > >> > > protocol at > > > > >> > > the moment, so it wont show your contacts, and the top info is > > > > >faked. > > > > >> > > But > > > > >> > > > > > >> > you > > > > >> > > > > > >> > > can see the graphical niceness of it...) > > > > >> > > > > > > >> > > Cheers, > > > > >> > > Tom > > > > >> > > > > > > >> > > On 12/3/06, [EMAIL PROTECTED] <[EMAIL PROTECTED] > > > > >> > > > > >> > > > > > >> > wrote: > > > > >> > > > Yeah, think its referring to when dragging contacts. > > > > >> > > > > > > > >> > > > On 12/3/06, NoWhereMan <[EMAIL PROTECTED]> wrote: > > > > >> > > > > NoWhereMan wrote: > > > > >> > > > > > Boris Faure (aka billiob) wrote: > > > > >> > > > > >>> how can I see your progresses with the new CL ? I'm > > > > >> > > > > >>> curious > > > > >:D > > > > >> > > > > > >> > what > > > > >> > > > > > >> > > > > >>> do I have to type in the status log? > > > > >> > > > > >> > > > > >> > > > > >> If it makes you feeling coding : > > > > >> > > > > >>>> guiContactList::createCLWindow > > > > >> > > > > > > > > > >> > > > > > I guess it still won't, but who knows :D > > > > >> > > > > > > > > >> > > > > I see this todo: "scroll the canvas while dragging..." you > > > > >> > > > > mean > > > > >> > > > > this behaviour was _by design_ ?? > > > > >> > > > > I mean, I always believed the side-scroll-on-drag thing was > > > > >> > > > > due > > > > >to > > > > >> > > > > > >> > the > > > > >> > > > > > >> > > > > fact > > > > >> > > > > the old CL is a text widget. > > > > >> > > > > O_O I never liked that; or am I missing something? > > > > >> > > > > > > > > >> > > > > If I were in you I would just drop that, and trim the > > > > >overflowing > > > > >> > > > > > >> > text > > > > >> > > > > > >> > > > > (of > > > > >> > > > > the PSM) with an ellipsis (...) as you do with the nick; all > > > > >> > > > > the > > > > >> > > > > > >> > text > > > > >> > > > > > >> > > > > should > > > > >> > > > > *always* be bound to its container (even if it's just a > > > > >background > > > > >> > > > > image > > > > >> > > > > > > > > >> > > > > ;)). > > > > >> > > > > Complete text should appear *only* in the tooltip, as a text > > > > >> > > > > overflowing on > > > > >> > > > > the right and without an horizontal scrollbar in the window > > > > >looks > > > > >> > > > > > >> > bad > > > > >> > > > > > >> > > > > and > > > > >> > > > > weird. > > > > >> > > > > But please don't add the horizontal scrollbar for that, as > > > > >> > > > > IMO > > > > >it > > > > >> > > > > > >> > would > > > > >> > > > > > >> > > > > look > > > > >> > > > > even worse :D > > > > >> > > > > > > > > >> > > > > If you are talking about using drag-n-drop to re-sort your > > > > >contacts > > > > >> > > > > > >> > or > > > > >> > > > > > >> > > > > something, then forget about this comment :D > > > > >> > > > > > > > > >> > > > > bye > > > > >> > > > > > >> > > > > > >----------------------------------------------------------------------- > > > > >> > > > > > >> > > > >-- > > > > >> > > > > > > > > >> > > > > Take Surveys. Earn Cash. Influence the Future of IT > > > > >> > > > > Join SourceForge.net's Techsay panel and you'll get the > > > > >> > > > > chance > > > > >to > > > > >> > > > > > >> > share > > > > >> > > > > > >> > > > > your > > > > >> > > > > opinions on IT & business topics through brief surveys - and > > > > >earn > > > > >> > > > > > >> > cash > > > > >> > > > > > >> > > > > > >> > > > > > >http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVD > > > > >> > > > > > >> > > > >EV _______________________________________________ > > > > >> > > > > Amsn-devel mailing list > > > > >> > > > > Amsn-devel@lists.sourceforge.net > > > > >> > > > > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > > >> > > > > > >> > > > > > >------------------------------------------------------------------------- > > > > >> > Take Surveys. Earn Cash. Influence the Future of IT > > > > >> > Join SourceForge.net's Techsay panel and you'll get the chance to > > > > >share > > > > >> > your > > > > >> > opinions on IT & business topics through brief surveys - and earn > > > > >> > cash > > > > >> > > > > > >http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > > >> > _______________________________________________ > > > > >> > Amsn-devel mailing list > > > > >> > Amsn-devel@lists.sourceforge.net > > > > >> > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > > > > > > > > > > > > >------------------------------------------------------------------------- > > > > >Take Surveys. Earn Cash. Influence the Future of IT > > > > >Join SourceForge.net's Techsay panel and you'll get the chance to share > > > > >your > > > > >opinions on IT & business topics through brief surveys - and earn cash > > > > >http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > > >_______________________________________________ > > > > >Amsn-devel mailing list > > > > >Amsn-devel@lists.sourceforge.net > > > > >https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > > > > > > > > > > ------------------------------------------------------------------------- > > > > Take Surveys. Earn Cash. Influence the Future of IT > > > > Join SourceForge.net's Techsay panel and you'll get the chance to share > > > > your > > > > opinions on IT & business topics through brief surveys - and earn cash > > > > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > > _______________________________________________ > > > > Amsn-devel mailing list > > > > Amsn-devel@lists.sourceforge.net > > > > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > > > > > > > ------------------------------------------------------------------------- > > > Take Surveys. Earn Cash. Influence the Future of IT > > > Join SourceForge.net's Techsay panel and you'll get the chance to share > > > your > > > opinions on IT & business topics through brief surveys - and earn cash > > > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > > _______________________________________________ > > > Amsn-devel mailing list > > > Amsn-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > > > > > ------------------------------------------------------------------------- > > Take Surveys. Earn Cash. Influence the Future of IT > > Join SourceForge.net's Techsay panel and you'll get the chance to share your > > opinions on IT & business topics through brief surveys - and earn cash > > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > > _______________________________________________ > > Amsn-devel mailing list > > Amsn-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/amsn-devel > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share your > opinions on IT & business topics through brief surveys - and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > Amsn-devel mailing list > Amsn-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/amsn-devel ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Amsn-devel mailing list Amsn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/amsn-devel
