[Kst] Feedback from testing Kst-2.0.x-Qt5-2012.10.17-07.54

[Peter Kümmel]

On 17.10.2012 12:05, Brisset, Nicolas wrote:
> Hi Peter,
>
> I've been doing some tests with the snapshot Kst-2.0.x-Qt5-2012.10.17-07.54 and here are some comments :
>
> -          When trying to load too much data without setting the file buffer limit, it does not crash and allows to try and continue in a dialog you added recently: very nice. Maybe we should mention the "Limit file buffer" option in this dialog so that people understand/discover how to work around the issue

OK, added the hint.

>
> -          I don't know what we do in the ascii config dialog, but it takes ages to get out of it when you've changed the settings and hit OK. Maybe worth a check?

It parses the file again for rows. And as I understand it the file is always read twice, first for wuilding the index , 
then when samples are read.

>
> -          Using threads on a single file now does bring some benefit. With a 1 GB file and 200 MB buffer I get 16.2 s (debug log) without threading and 10.5 s with threading. But the difference from the user perspective as measured by the "wall clock" is not as large (19 s vs 14 s), which is in fact quite understandable since all the drawing is common
>
> -          I tried to change the "use threads" setting on some (by hand-editing the .kst file) and it seems to bring a real benefit => shouldn't we activate it by default instead of defaulting to false (which actually is more or less what I wrote in the previous mail: why not thread in all cases?

This feature is so new, that I'm not sure we should enable it by default.

>
> -          I have some files which seem to open with 2.0.6 and not with this snapshot (or even crash). I'll investigate this a bit further. Could you maybe generate a beta with Qt4 and MSVC to make sure we're not hitting a bug there?

Yes, I will upload it to SF.

>
> To sum up: great work!
> I'll do some more testing (and go into the details of indexes etc) and I hope we can settle on the UI aspects/defaults quickly to finalize this release, hoping that the problems with old files I see are linked to Qt5 / mingw versions in the snapshot.
>
> Nicolas
>