Issue 67271 - accelerate chart render/layout ...
Summary: accelerate chart render/layout ...
Status: CLOSED WONT_FIX
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: bjoern.milcke
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-12 12:47 UTC by mmeeks
Modified: 2013-02-24 21:19 UTC (History)
3 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
patch (4.96 KB, patch)
2006-07-12 12:48 UTC, mmeeks 		no flags Details | Diff
slow .xls file (840.00 KB, application/vnd.ms-excel)
2006-07-14 15:30 UTC, mmeeks 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description mmeeks 2006-07-12 12:47:53 UTC 
So - this gives a ~5 fold improvement for the attached spreadsheet wrt. laying
out it's rows.

The 1st 1/2 is (I hope) uncontroversial, the 2nd - does a 'same-but-for-digits'
string-compare, which assumes that all digits are the same width: incorrect, but
near enough :-) [ perhaps we should only do that for > <N> labels ].

Either way - with these, the attached sheet loads / renders in 10s of seconds
instead of minutes.

HTH.
Comment 1 mmeeks 2006-07-12 12:48:18 UTC 
Created attachment 37712 [details]
patch
Comment 2 kla 2006-07-12 13:42:39 UTC 
kla->bm: Maybe one for you.
Comment 3 mmeeks 2006-07-12 15:13:01 UTC 
and of course in manually editing a conflict I forgot to append a ')' to these
lines:
if(rString.Len() > 0 && (mbTextOverlap || DetectOverlap(rString, aPos, eAdj))
sorry.
Comment 4 bjoern.milcke 2006-07-12 16:59:46 UTC 
Will check if this works.
Comment 5 mmeeks 2006-07-14 15:30:02 UTC 
Created attachment 37770 [details]
slow .xls file
Comment 6 bjoern.milcke 2006-10-20 13:18:58 UTC 
->Michael: As already said in the IRC, I think this is too risky. You said, you
found problems in this version of the patch. I suggest to wait for the new chart
for performance improvements.
->Closing
Comment 7 bjoern.milcke 2006-10-20 13:54:25 UTC 
Closed.
Comment 8 mmeeks 2007-07-05 10:51:32 UTC 
This is in fact fixed really nicely in chart2 - loads like a (very short) dream :-)