Memoy leak using QtCharts

aawawa

Hello everyone,

I hope you are haveing a nice day,

I am currently trying to plot indefinitely a graph on a chartview.

I have memory leak that I can't find the origine.

Here is a quiet simple code to illustrate this fact :

void MyChartView::Draw(void)
{
	int Length = 10000;
	double Frequency = 0.1;
	chart()->removeAllSeries(); //here I remove the series
	QLineSeries *SerieToAdd = new QLineSeries;
	for (int i = 0; i < Length; i++)
	{
		SerieToAdd->append(i, sin(static_cast<double>(i) * 3.14*2*0.1)); 
	}
	chart()->addSeries(SerieToAdd); //here I add the serie
}

for information, in this example, MyChartView is a basic class, nothing really creazy:

class MyChartView:public QChartView
{
public:
	MyChartView();
	~MyChartView();
	void Draw(void);
};

If I launch 10000 times the function Draw like below, I will see a memory leak.

Do you know what I am doing wrong?

kuzulis

Do you know what I am doing wrong?

Just don't use QtCharts... Use, e.g Qwt.

mrjj

Hi
How big is this leak ?
Could you try to do
MyChartView test; ( as in no new.ing)
Currently the test is not deleted as you assign no parent.

aawawa

@kuzulis : hahahhabut by the way. I want to understand here why it doesn't work in this case!

@mrjj : something around 10ko for each time when Draw is used.

There is a strange thing. If I code the code below :

void MyChartView::Draw(void)
{
	int Length = 10000;
	double Frequency = 0.1;

	chart()->removeAllSeries(); //here I remove the series
	QLineSeries *SerieToAdd = new QLineSeries;
	for (int i = 0; i < Length; i++)
	{
		SerieToAdd->append(i, sin(static_cast<double>(i) * 3.14*2*0.1)); 
                          //or whatever
	}

	//chart()->addSeries(SerieToAdd); //here I comment this line

	delete SerieToAdd;

}

I don't have any memoy leak. But, if I wrote the same code but with the addSeries line, the leak appear... (here below)

void MyChartView::Draw(void)
{
	int Length = 10000;
	double Frequency = 0.1;

	chart()->removeAllSeries(); //here I remove the series
	QLineSeries *SerieToAdd = new QLineSeries;
	for (int i = 0; i < Length; i++)
	{
		SerieToAdd->append(i, sin(static_cast<double>(i) * 3.14*2*0.1));
 //or whatever
	}
	chart()->addSeries(SerieToAdd); //here I add the serie!!!!!!!LEAK
	delete SerieToAdd;
}

This is quiet strange because I am supposed to remove and to delete the series that I have added to the chart at each iteration with the command removeAllSeries() no? So normally the size of my object must be constant in time...

Could you try to do MyChartView test; ( as in no new.ing) I am sorry i m not sure that I have understand what you asked me...

Currently the test is not deleted as you assign no parent. Yes! I don't want to delete the test, I just want that its size remains constant in time...

mrjj

Hi

Currently the test is not deleted as you assign no parent. Yes! I don't want to delete the test, I just want that its size remains constant in time...

I understand. i was just wondering if the lost memory was due to some house keeping in ChartView
that would only be freed on app exit./ Widget destruction.

However, it does seem odd as removeAllSeries says it will delete it and
addSeries says it takes ownership so all should be fine.

Could you try rem out
// SerieToAdd->append(..
and see if mem use is affected. Then we know its directly related to that.

Christian Ehrlicher

A small, compilable testcase would be the best here so we can check it (and maybe at it to the bugreport later on) :)

kuzulis

@aawawa said in Memoy leak using QtCharts:

hahahhabut by the way. I want to understand here why it doesn't work in this case!

I'm too tried to use QtCharts and too faced with a memory leaks and etc.. I spat on QtCharts and replaced it with Qwt. (Because I haven't time to understand why QtCharts does not work as expected.. This was in Qt 5.9, as I remember).

kshegunov

@kuzulis said in Memoy leak using QtCharts:

I spat on QtCharts and replaced it with Qwt. (Because I haven't time to unterstand why QtCharts does not work as expected.. This was in Qt 5.9, as I remember).

It hasn't improved since then. I did the same as you but picked QCustomPlot instead.

aawawa

@Christian-Ehrlicher : please find a micro project here : link text

@mrjj : if I comment as this :

//SerieToAdd->append(i, sin(static_cast<double>(i) * 3.14*2

I have always the memory leak but it seems to be less important (around 6ko)

mrjj

@aawawa
Ok so it does seems related to the series.
If you want to dig into it, you can very handy browse code here
https://code.woboq.org/qt5/

Super with sample.

aawawa

@mrjj I 'm going to take a look to your link. Is it a database or you want me to send my code there?

mrjj

@aawawa
Hi
Its a code browser. it makes it really easy to see what is being used. all call to functions are links and
its almost as good as the debugger. at least to see what is going on.
I was thinking you would go and look how it deletes the series or something like that.
The sample is for Christian-Ehrlicher as he might make it into a bug report if we can
reproduce it and find no good reason for not release the mem on RemoveAllSeries.

Christian Ehrlicher

I can see some kind of 'leak' but need to do some more investigations.

Christian Ehrlicher

Ok, no leak there. Just a lot of data. You're creating 100000 QLineSeries with 10000 each - this will need some memory and time. I would rather try to add the points to one QLineSeries.

aawawa

@Christian-Ehrlicher thank you... But it is supposed to be deleted 100000 times in my case no when I use RemoveAllSeries no?

aawawa

@Christian-Ehrlicher would rather try to add the points to one QLineSeries this strategy works quiet well.
By changing y program as this for the header :

class MyChartView:public QChartView
{
public:
	MyChartView();
	~MyChartView();
	void Draw(void);

private:	
	QLineSeries *mSeries;
};

and for the .cpp

MyChartView::MyChartView()
{

	mSeries = new QLineSeries;
	chart()->addSeries(mSeries);

}


MyChartView::~MyChartView()
{
}

void MyChartView::Draw(void)
{
	int Length = 1000;
	mSeries->clear();
	for (int i = 0; i < Length; i++)
	{
		mSeries->append(QPoint (i, sin(2 * i*3.141516*0.1)));
	}
	//update();

}

It seems (I have to confirm this point later) to not have anymore memory leak

Christian Ehrlicher

@aawawa said in Memoy leak using QtCharts:

But it is supposed to be deleted 100000 times in my case no when I use RemoveAllSeries no?

You're right. I think I found the culprit:

qlineseries.cpp:112

QLineSeries::QLineSeries(QObject *parent)
    : QXYSeries(*new QLineSeriesPrivate(this), parent)

this is never deleted... :/

JonB

@Christian-Ehrlicher
I think this is the second time I've seen that *new cause this. When will people decide that C++ is getting too clever for its boots? ;-)

SGaist

Hi,

@Christian-Ehrlicher said in Memoy leak using QtCharts:

QLineSeriesPrivate

~~It's QObject based, so when the QXYSeries object is deleted it should also get deleted.~~

It's indeed QObject based but stored in a QScopedPointer so it will automatically get delete on object's destruction.

aawawa

@Christian-Ehrlicher Thank you. It's the first time I see *new in c++ what does it mean?