Ok, I tested it on my current developing build, which is branched off of master from 2019-04-29 (a0ba9d263118bc2df2aa922c124c3db57acbd2dc). I can reproduce the error as described in the description. So I guess to answer your question @kislinsk: The status is "still open".

I forgot to enable the segmentation plugin... I'll test it tomorrow.

I can reproduce it. I did exactly the steps in the task description. I used MITK.2018.4.0 installer. I#ll check on a (more or less) current master.

Wait for it...

Pushed new branch T25683-chart_example_linux.

Nice!

I do not think, that the subchart feature is a blocker or even critical. It was very easy to implement with C3 and I personally liked the idea as described above. I assume there is something to achieve that in plotly as well? I'd be happy to discuss this with you in person.

The idea behind that feature was, that it should be obvious to the user, that he is only looking at a part of the data. A zoomed view can lead to misinterpretation if the user is not aware of it. At least that is what I thought when I added that feature.

I created a subtask. We'll look into this.

We just took a look at plotly.js. In contrast to C3, it supports error-bars and is still under development (see guithub stats).
To use plotly.js with MITK chart we think, that no fundamental changes on C++ side are necessary. Just the JS stuff needs to be rewritten.

Try: E130-Daten\BlackSwan\Patients_for_Registration\0HLXYUL7_0HLXYUL7\16\resources\DICOM

There might also be a Problem when changing the theme. You might have to deselect and reselect the data node. But generally speaking the histogram is available in dark, it works and looks neat.

see http://docs.mitk.org/nightly/NewPluginPage.html
Spaces are ok, but we should test it one more time.

Also works for me. I'll close this task.

Instead of hiding the line one could just use a scatter plot... I will look into this next time

@vemuri and I integrated the proposal by @laha and @hempe.
The solution needs to be tested with multiple data lines. We tested it within the Image statistics plugin and it worked.

Pushed new branch T23468-add-line-style-hidden.

Yes

Pushed new branch T23912-histogram-background-color.

We identified the problem.

• default style in QtStyleManager does not change the preference
• preference page does not know about the default style
• preference page clears the QtStylemanager and thus resets to default each time

Can you explain how to reproduce this? We tried it on a fresh Superbuild of the latest master today and it worked as expected.
@kalali @kahl

+1

Added chart type as a data property.

• This is needed to be able to display multiple chart types at the same time. (e.g. in MitkDiffusion)
• Therefore we must get rid of the "global" chart type.
• Chart type is now passed as a parameter in AddDataXd methods

To bad :-( So I guess we'll stick to C3.js for now.

Makes sense to me. Labels are now stored as Property in QmitkChartxyData. Therefore the Property mDataLabels of QmitkChartData has been removed.

Pushed new branch T23472-areaSplineMitkChart.

Pushed new branch T23467-areaChartMitkChart.

Done.

Pushed new branch T23463-change-bg-clolor-for-MitkChart.