Re: [Gimp-developer] scale-region.c
On Mon, 2008-08-25 at 09:46 +0200, Sven Neumann wrote: [...] > This is just a bug in the progress code in scale-region.c. I am on it, > should be fixed by tonight. Oh you sexy bean! Now that you ahve indeed stopped gimp from hanging... timings for the 13818x8480 image are scaling down to 50%: 6 seconds (was 30 seconds a few days ago) scaling down to 51%: 6 seconds (was 41 seconds a few days ago) Liam -- Liam Quin - XML Activity Lead, W3C, http://www.w3.org/People/Quin/ Pictures from old books: http://fromoldbooks.org/ Ankh: irc.sorcery.net irc.gnome.org www.advogato.org ___ Gimp-developer mailing list Gimp-developer@lists.XCF.Berkeley.EDU https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer
Re: [Gimp-developer] scale-region.c
Hi, On Sun, 2008-08-24 at 15:27 -0400, Liam R E Quin wrote: > By "freeze-at-end" I mean that the progress bar stops moving > when it is almost at the very end; my guess is that it's > pushing onto the undo stack and also maybe generating a > thumbnail for the undo history, but that's an uninformed > guess :-) This is just a bug in the progress code in scale-region.c. I am on it, should be fixed by tonight. Sven ___ Gimp-developer mailing list Gimp-developer@lists.XCF.Berkeley.EDU https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer
Re: [Gimp-developer] scale-region.c
On Sun, 2008-08-24 at 19:09 +0200, Sven Neumann wrote: [...] > Thanks for the review. I have committed this change and some other > cleanups and optimizations to SVN trunk last night. This gives a small > but noticeable speedup. I hope that my changes did not introduce any new > bugs, but I am quite confident that I haven't broken anything. Some informal (approximate) timings with a grayscale image, 13818x8480 pixels (a sketch by Sydney Jones). before patch: scale 50% = 18 seconds to freeze-at-end, 31 secs overall 17 seconds to freeze-at-end, 29 secs overall 51% = 14 seconds to freeze-at-end, 43 secs overall 11 seconds to freeze-at-end, 40 secs overall after patch: scale 50% = 07 seconds to freeze-at-end, 24 secs overall 07 seconds to freeze-at-end, 25 secs overall 51% = 09 seconds to freeze-at-end, 34 secs overall 10 seconds to freeze-at-end, 35 secs overall I did the timings more than once. I have 8G of RAM and a 7G tile cache size (on Sven's suggestion - it does seem to speed things up) By "freeze-at-end" I mean that the progress bar stops moving when it is almost at the very end; my guess is that it's pushing onto the undo stack and also maybe generating a thumbnail for the undo history, but that's an uninformed guess :-) The speedup for the first part is very noticeable, e.g. 18 secs -> 7 secs, but it makes the "frozen" part more noticeable, and as a result actually seems slower. I did not do regression testing, comparing the results, sorry. Liam -- Liam Quin - XML Activity Lead, W3C, http://www.w3.org/People/Quin/ Pictures from old books: http://fromoldbooks.org/ Ankh: irc.sorcery.net irc.gnome.org www.advogato.org ___ Gimp-developer mailing list Gimp-developer@lists.XCF.Berkeley.EDU https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer
Re: [Gimp-developer] scale-region.c
Hi, On Sun, 2008-08-24 at 14:59 +0200, Geert Jordaens wrote: > I see no problem with that,it should be safe. Thanks for the review. I have committed this change and some other cleanups and optimizations to SVN trunk last night. This gives a small but noticeable speedup. I hope that my changes did not introduce any new bugs, but I am quite confident that I haven't broken anything. Sven ___ Gimp-developer mailing list Gimp-developer@lists.XCF.Berkeley.EDU https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer
Re: [Gimp-developer] scale-region.c
Sven Neumann wrote: > Hi Geert, > > I have a small patch to scale-region.c that I would to have your opinion > on. I noticed that the current code sometimes does an unneeded copy > operation. This happens when the scale factor is 2^n. For example when > an image of 800x600 pixels is scaled to 400x300. The function > determine_scale() then decides that a first scale step to 400x300 needs > to be made and the result is then scaled again with a scale factor of > 1.0. There's even an optimization n the scale() function for this > special case. > > Attached is a patch that changes determine_scale() to avoid this extra > step. Is there anything that I am missing or do you agree that it should > be safe to apply this? > > > Sven > > > > > > ___ > Gimp-developer mailing list > Gimp-developer@lists.XCF.Berkeley.EDU > https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer I see no problem with that,it should be safe. ___ Gimp-developer mailing list Gimp-developer@lists.XCF.Berkeley.EDU https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer