We should switch from QmitkPlotWidget to MitkChart now
Description
Description
Status | Assigned | Task | ||
---|---|---|---|---|
Wontfix | None | T23104 Deprecate QmitkPlotWidget | ||
Resolved | kislinsk | T25832 Migrate ModelFit inspector plugin to MitkChart | ||
Open | floca | T26366 [ModelFitInspector] Fixed scaling doesn't work yet | ||
Open | floca | T26367 [ModelFitInspector] Remove bookmark doesn't work | ||
Open | floca | T26368 [ModelFitInspector] Performance if multiple bookmark points are added |
Event Timeline
Comment Actions
If you have already an interim branch, could you push it, just to be sure that it doesn't get lost.
Thanks.
Comment Actions
Feedback from @floca:
- Der Titel der X-Achse und die Achsenbeschriftungen liegen über einander und sind deshalb nicht zu lesen.
- is already fixed with T26274, but not yet in this branch
- Fixed Scaling (für X und Y) funktioniert nicht richtig.
- true
- In der Methode OnFullPlotClicked ist ein Task (T26222) referenziert, den es gar nicht gibt. Vielleicht ist das aber jetzt auch schon durch meinen Fix erledigt.
- code was deleted, works fine
- Kritische Regression gegenüber dem alten Widget: Im Moment flackert es immer sobald man die Achsen anpasst oder (noch schlimmer) wenn man einen neuen Punkt ansteuert. Dass ist insbesondere beim explorieren von Daten nicht gut, da es leicht laggt und sehr irritiert.
- opened T26320
- Kritische Regression gegenüber dem alten Widget: Im Moment aktualisiert sich der Plot erst wenn man die Maus wieder loslässt (wenn man im SliceNavigator oder direkt im Bild neue Punkte anfährt). Früher wurde automatisch immer gleich mit aktualisiert, es wäre sehr gut, wenn das wieder so wäre. Entsprechend müsste der Chart neugemalt werden, jedesmal wenn die Positionsänderung propagiert wird. Im Moment sieht es so aus, als würde gewartet, bis der Main-Thread idelt.
- can't reproduce. The update takes a while (probably because of new painting of the widget), but has nothing to do with mouse click release. @floca we should look again if T26320 is fixed.
Comment Actions
Unclaime as I currently have no time to work on it. Will take on it again, if I have time or some else;)
Comment Actions
Should be done after(!) we have moved to Qt6 as we then have QtCharts to our disposal and migrate to that.
Comment Actions
This task was closed here on Phabricator since it was migrated to GitLab. Please continue on GitLab.