tech-radar icon indicating copy to clipboard operation
tech-radar copied to clipboard

add zoom to scatterplot

Open Kvaz1r opened this issue 6 years ago • 5 comments

For #84 .

There isn't anything really new - all changes just copy-pasted code from wxmath2dplot.

Kvaz1r avatar Jan 17 '19 14:01 Kvaz1r

Any plans to merge these changes? :)

Anton-V-K avatar May 26 '19 13:05 Anton-V-K

I tried the code-changes (from thefile-changes) but it failed with shredptr.h(85): assert "m_ref!=_null" failed in operator ->0

I try to minimize the xy-dots to smaller points from the size like ball's (as it is now)

helha54 avatar Mar 07 '21 10:03 helha54

I tried the code-changes (from thefile-changes) but it failed with shredptr.h(85): assert "m_ref!=_null" failed in operator ->0

I try to minimize the xy-dots to smaller points from the size like ball's (as it is now)

  1. Code in this PR doesn't contains anything that can lead to break not null assert. I think that reason the same as in #165. 3 datasets is current restriction for all charts.

  2. As for now radius for scatter always equals 5, but you can change the value here and re-build the library.

Kvaz1r avatar Mar 07 '21 13:03 Kvaz1r

I found the reason. There are two lines missing in, when re-insert them everything is ok. That is one include-line #include "wxchartstheme.h"

wxScatterPlot::wxScatterPlot(const wxScatterPlotData &data, const wxSize &size) :m_options(wxChartsDefaultTheme->GetScatterPlotOptions()),

where the last line ' : m_opions' is missing

After inserting them everything is ok.

helha54 avatar Mar 07 '21 13:03 helha54

I reduced the point-size to 1 and that is what i needed.

I run scatterplot with 5 datasets, so it seems to work with 5 datasets least. ( just as information)

helha54 avatar Mar 07 '21 14:03 helha54