[Kst] Re: branches/work/kst/portto4/kst/src

[Peter Kümmel]

On 21.01.2011 18:36, Brisset, Nicolas wrote:
>> SVN commit 1216165 by kuemmel:
>>
>> Some ascii reading optimization:
>>
>> Here some measurments: gyrodata c&p to 8 million rows
>>
>> Delimiters (comment, column)      : time in readColumns
>> -----------------------------------------------------------
>> No comment, custom: one space     : 1.69102 seconds
>> No comment, whitespace            : 2.11194 seconds
>> default comment, custom: one space: 4.01525 seconds (default comment
> with
>> 4 characters)
>> default comment, whitespace       : 4.34039 seconds
>> default comment, custom ,;|       : 6.49383 seconds (space at the end
> of 4
>> characters)
> Thanks Peter! This is very interesting and of big practical interest.
>
> A couple of questions:
> - does the overcost of comments increase with the number of symbols? I'd
> like to remove :/; and leave only # in the default comments because of
> all the confusion it creates =>  what would the result be with only "#"
> as comment indicator? (If it's a big penalty, we could leave the field
> empty by default and put some explanations in the tooltip)

It depends on the implementation of QString::contains:

using custom column 1 space
0  symbols: 1.7
1  symbols: 1.8 #
2  symbols: 3.8 ab
4  symbols: 4.0 #/c!;
10 symbols: 4.3 abcdefghij
10 symbols: 5.1 ;:=()$§%?#

using whitespace
0  symbols: 2.09
1  symbols: 2.20 #
4  symbols: 4.40 #/c!;

> - how does it compare to the situation before the last changes you made?
> (In other words, is the speed still the same as a couple of days ago, or
> have you been able to improve it further)?

See my last commit.
Maybe we could improve performance more by aggressive inlining.

>
> Nicolas
>
>
> Eurocopter Deutschland GmbH
> Sitz der Gesellschaft/Registered Office: Donauwoerth
> Registergericht/Registration Court: Amtsgericht Augsburg HRB 16508
> Vorsitzender des Aufsichtsrates/Chairman of the Supervisory Board: Dr. Lutz Bertling
> Geschaeftsfuehrung/Board of Management:
> Dr. Wolfgang Schoder, Vorsitzender/CEO; Friedrich-Wilhelm Hormel; Ralf Barnscheidt
>
> CONFIDENTIALITY NOTICE
>
> This communication and the information it contains is intended for the addressee(s) named above and for no other persons or organizations. It is confidential and may be legally privileged and protected by law. The unauthorized use, copying or disclosure of this communication or any part of it is prohibited and may be unlawful.
> If you have received this communication in error, kindly notify us by return e-mail and discard and/or delete the communication. Thank you very much.
> It is possible for e-mails to be intercepted or affected by viruses. Whilst we maintain virus checks on our e-mails, we accept no liability for viruses or other material which might be introduced with this message.
>
> _______________________________________________
> Kst mailing list
> Kst at kde.org
> https://mail.kde.org/mailman/listinfo/kst
>