Modify the GUI in order to make it more intuitive. Also adding some new features such as updating existing data.
Additional todos:
T26069: Logarithmic axis scaling display problems
T26070: Stacked Data ignores set chart type
{T25896}
Modify the GUI in order to make it more intuitive. Also adding some new features such as updating existing data.
Additional todos:
T26069: Logarithmic axis scaling display problems
T26070: Stacked Data ignores set chart type
{T25896}
I started reviewing the plugin.
Things I found will be directly mentioned in comments in your commits.
However, there are some things that I would like you to check which were not written by you. In order to finalize the refactoring of this plugin, it makes sense to change these things as well:
Please make sure to have newlines at the end of your source code files (see https://stackoverflow.com/questions/5813311/no-newline-at-end-of-file).
Sooner then later I have to restart on reworking the ChartWidget due to performance problems when transfering data to the java script side. Currently each line is communicated on its own and this communication has an overhead. Paied for every graph line on an update it is a serious performance bump that impacts usability e.g. in the perfusion context.
Do you know who long your rework in the ChartWidget will take? If possible I would like to avoid to parallel developments and then having a forseeable merge trouble.
We want to merge the recent changes into master this week. Most changes were done inside the chartexample-plugin but we will update this task as soon as our work here is done.
Hi there! 🙂
This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑
Best wishes,
The MITK devs