search

seantronsen / pvt

GNU General Public License v3.0
0 stars 0 forks source link

Refactor: Separate Plot2D #9

Closed seantrons closed 6 months ago

seantrons commented 6 months ago

Currently, Plot2D is used to draw both line and scatter plots onto the same display. The current implementation doesn't even allow for the two to be plot simultaneously on the same data display pane (only support for multiple plots of the same type). These should be refactored into unique classes that handle these responsibilities separately.

At some point in the future, implementing another level of abstraction may be desirable to allow for both types to be plot on the same pane simultaneously.

seantronsen commented 6 months ago

Refactoring complete. Working on documentation comments.

seantronsen commented 6 months ago

The functionality of the original class has been separated into Plot2DScatterPane and Plot2DLinePane.

Updates can be found in this commit: https://github.com/seantronsen/qtviewer/commit/0492178ec8726757b434c8ce3ccf1902fde82dbe

Closing.