Issue 65549

Summary: Plotting of missing values (leave gap,assume zero,continue line)
Product: General Reporter: IngridvdM
Component: chartAssignee: kla <thomas.klarhoefer>
Status: CLOSED FIXED QA Contact: issues@graphics <issues>
Severity: Trivial    
Priority: P3 CC: bjoern.milcke, issues, kamataki, pagalmes.lists, tony.galmiche.ooo, weizhao
Version: 3.3.0 or older (OOo)Keywords: ms_interoperability, rfe_eval_ok
Target Milestone: ---   
Hardware: All   
OS: All   
URL: http://specs.openoffice.org/chart/MissingValues.odt
Issue Type: FEATURE Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on: 91899    
Issue Blocks: 64819, 72764, 86346    
Attachments:
Description Flags
patch_080110
none
patch_080129
none
TCS_Plotting_of_missing_values none

Description IngridvdM 2006-05-18 16:14:18 UTC
If a data range for a series does contain empty cells one can think of different
ways to handle those. For example:
a) don't pain't
b) assume zero
c) interpolate value
It should be possible for the user to choose which handling he would like to use.

Further details have to be specified, e.g.
-what about invalid values
-empty values in x-ranges
-zero option in logarithmic charts
-maybe an automatic flag to handle the different default behaviours of scatter
and line charts (Line Chart->don't paint, Scatter->Interpolate).
Comment 1 IngridvdM 2006-05-18 16:15:06 UTC
accepted
Comment 2 IngridvdM 2007-10-08 10:16:37 UTC
->bm, please take over.
Comment 3 IngridvdM 2007-10-25 07:55:44 UTC
changing target due to limited resources
Comment 4 IngridvdM 2007-11-07 19:36:50 UTC
*** Issue 74204 has been marked as a duplicate of this issue. ***
Comment 5 bjoern.milcke 2007-12-18 08:53:30 UTC
For the file-format specification see
http://lists.oasis-open.org/archives/office/200702/msg00009.html
Comment 6 bjoern.milcke 2007-12-19 12:59:51 UTC
About placing the UI controls for this:

I think there are basically two places where the controls could be placed.

1. In the chart type dialog (and therefore on first page of the wizard). This is
a property at the diagram, and the chart type is the closed choice. It would
have to appear on all chart-type option panels for the following types: line,
scatter (only if points are connected?), area, net, but also bar/line combi chart.

Due to limited space on that pages I would suggest to use a different place:

2. The options tab page of a data series. It is not ideal, but conforming to the
chart-type specific controls like gap-width, overlap and "bars side-by-side" for
bar and column charts. There is enough space to add radio buttons and maybe
additional check boxes.
Comment 7 weiz 2008-01-11 01:31:25 UTC
->bm, iha: The attached is the patch of this issue, I think it's not the final 
patch, I just want to show you what I have done, and to confirm whether it's 
correct or not. I will change or improve it according to your suggestions. :-)
Comment 8 weiz 2008-01-11 01:32:22 UTC
Created attachment 50794 [details]
patch_080110
Comment 9 IngridvdM 2008-01-14 17:34:06 UTC
->weiz: The things written to the files need to be changed. Please look at the
file format definition:
http://lists.oasis-open.org/archives/office/200702/msg00009.html
The attribute is defined to have the name 'treat-empty-cells' not 'nan-data-handle'.
Furthermore the new token is not of type number but instead three different
string should be written to the file, dependent on the choice:
'use-zero' or 'leave-gap' or 'ignore'. As an example look at the token
'label-position'.

Regarding the UI strings I would call the second radio button 'Use zero' or
'Assume zero' or similar. Thus it is more consistent with the other descriptions.

Please try to move the new controls upwards for line charts. It looks very
strange at the moment with that much unused space in between. You can change the
positions of the controls also from c++ code. Please try to move them up
depending on the show/hide behavior of the previous controls in
SchOptionTabPage::Reset.

There needs to be done something around the default values also I think. It is
not so nice to have nothing checked when opening the dialog. Have a look how
this is handled for the new label placements
ChartTypeHelper::getSupportedLabelPlacements as an example.
Comment 10 weiz 2008-01-29 05:20:55 UTC
->iha, bm: The new patch is finished, it's beased on m243. Please let me know 
your suggestions. Thanks a lot!
Comment 11 weiz 2008-01-29 05:22:04 UTC
Created attachment 51217 [details]
patch_080129
Comment 12 IngridvdM 2008-02-20 16:51:06 UTC
->weiz: Thanks for the patch! I've looked at it and did some additional changes
during your vacation as communicated via mail. When the previous chart feature
CWS chart19 is integrated I will create a new CWS chart22 based on that new
master and check in your patch including my additional changes.
Comment 13 IngridvdM 2008-02-22 10:33:57 UTC
Fixed in CWS chart22. For the details have a look at the specification:
http://specs.openoffice.org/chart/MissingValues.odt
The im/export to xcel wil be implemented with issue 86346.
->Thomas, please verify.
Comment 14 kla 2008-04-15 16:05:42 UTC
Created attachment 52828 [details]
TCS_Plotting_of_missing_values
Comment 15 kla 2008-04-16 13:30:38 UTC
Seen ok in CWS chart22 -> verified
Comment 16 IngridvdM 2008-07-11 09:32:43 UTC
The integration of this fix to the OOo 3.0 code line was refused so I change the
target to OOo 3.1.
Comment 17 IngridvdM 2008-09-16 13:45:13 UTC
*** Issue 3411 has been marked as a duplicate of this issue. ***
Comment 18 IngridvdM 2008-09-30 15:01:56 UTC
*** Issue 94365 has been marked as a duplicate of this issue. ***
Comment 19 kla 2008-12-22 09:12:18 UTC
Seen ok in current master(m37) -> closed