Hello, I have also written a MCTS, but it can only simulate 10W times, and then the memory leak will occur. Is there any way to solve it? about othellomcts-dataoriented HOT 16 CLOSED

may I see your code?

from othellomcts-dataoriented.

could you send me e-mail address?I can send to you.Thank you.

from othellomcts-dataoriented.

[email protected]

from othellomcts-dataoriented.

already send.Thank you.

from othellomcts-dataoriented.

see if this helps:
qDebug()<< "MCTS_Nodes"<< node_count<<"剩余落子点数:"<<best_child->maxChildrenNum;
delete node; //here

from othellomcts-dataoriented.

It does not work.the same problem.

from othellomcts-dataoriented.

then maybe give it a destructor
Node::~Node()
{
for(auto child :children)
delete child;
}

from othellomcts-dataoriented.

It can not add the fuction

from othellomcts-dataoriented.

then we work around it

void Node::delNode()
{
    for (auto child : children)
        child->delNode();
    delete this;
}

qDebug() << "MCTS_Nodes" << node_count << "剩余落子点数:" << best_child->maxChildrenNum;
node->delNode();

from othellomcts-dataoriented.

It already does not work.

from othellomcts-dataoriented.

well if you can't delete it don't allocate the node yourself.

from othellomcts-dataoriented.

I tried to delete the nodes, but it didn't work, and Monte Carlo searches would not work if no nodes were allocated.

from othellomcts-dataoriented.

I try again ,find it will be crashed whatever times I set simulation.

from othellomcts-dataoriented.

No. what I meant is don't allocate them yourself

Store the children in, say, a vector. Avoid using thing like "node newnode = new node".
something like

node newchild
//do your expand here
children.push_back(newchild)

from othellomcts-dataoriented.

this way the default constructor will deallocate the children for you.

from othellomcts-dataoriented.

I have a try

from othellomcts-dataoriented.