QGriddlayout works for grid 80*80 but not for 100*100 or more (SIGSEGV error)
-
-
@Davay confirm that as you increase the grid from 80x80 to 100x100, the following index access from the matrix is valid?
(*this)(r,c).getState()
(*this)(r,c).getMaterial()
(*this)(r,c).getBurningTime()My guess is that matrix is being initialized to 80x80
-
@Davay said in QGriddlayout works for grid 80*80 but not for 100*100 or more (SIGSEGV error):
Can you elaborate why it would lead to problems
You call the eventloop but it's not run in the 'outer' loop. So basically you have
outer main eventloop -> your program logic -> inner event loop
but this can lead to problems, especially when your program logic was called as a result of a queued signal/slot connection. When you then call the eventloop inside this slot, the signal may be evaluated again since it's not yet finished. Also the order of the signals can be mixed up due to this and other stuff which is very hard to debug. Therefore avoid it, especially when you try to delay something. Qt is event driven and you should not do such things.
and why to problems only occur for big enough grids?
timing or others, no idea but what you're doing is not recommended.
@Christian-Ehrlicher Thank you for the elaboration! I've followed your advice and it did help me with some laggy issues of my program but sadly not with the grid. I've changed my delay function in:
QTimer ts; ts.start(200); QEventLoop loop; QObject::connect(&ts, SIGNAL(timeout()), &loop, SLOT(quit())); loop.exec(); -
Sorry in advance I am very new to QT and c++. I am trying to make a grid filled with colored clickable labels. And the whole program works perfectly for grids the size smaller than 80*80 but for bigger gridds I get a SIGSEGV segmentation fault error in the debugger, it doesn't exactly tell my why. This is the code where the error most likely originates:
void Gridd::ShowStart(QWidget *wid){ QGridLayout *layout = new QGridLayout; for (unsigned int r = 0; r < rows; r++) { for (unsigned int c = 0; c < cols; c++) { ClickableLabel *label = new ClickableLabel(r,c); QString str = " background-color : " ; if ((*this)(r,c).getState() == "fireless"){ str += (*this)(r,c).getMaterial().getColor(); } else { int bt = (*this)(r,c).getBurningTime(); QColor col (250-20*bt,(80+bt * 160)/(1+bt*bt),0); } label->setStyleSheet(str); layout->addWidget(label, r, c); str=""; (*this)(r,c).setLabel(label); } } layout->setHorizontalSpacing(0); layout->setVerticalSpacing(0); wid->setLayout(layout); wid->show(); }I think it originates from here because it does not want to show my start grid here. Thanks in advace!
@Davay said in QGriddlayout works for grid 80*80 but not for 100*100 or more (SIGSEGV error):
I get a SIGSEGV segmentation fault error in the debugger, it doesn't exactly tell my why.
I don't see anywhere you have looked at/shown the stack trace for this?
-
@Davay said in QGriddlayout works for grid 80*80 but not for 100*100 or more (SIGSEGV error):
I get a SIGSEGV segmentation fault error in the debugger, it doesn't exactly tell my why.
I don't see anywhere you have looked at/shown the stack trace for this?
-
@JonB Thank you for your reply, I hope this picture helps but I wasn't really sure what you've asked for (since I'm really unfamiliar with qt (and c++).
The first picture is just before I get the Sigsev error and this second one is just after: 
-
The debug output looks like you have infinite recursion. With every function call you allocate more stack space. Eventually you will run out of stack space (as it is quite limited compared to the heap) which will in turn result in a segmentation fault.
Like @JonB already said, you need to provide more of the stack trace until you reach your own code. Click on <More> in the stack trace until you see something familiar. You should also figure out if Qt uses more than 1 Thread. If I am not mistaken GUI stuff runs in Thread
#0and not#1like you show it. (I might be mistaken about this, though.) Sometimes switching the thread will show your own code in the stack trace. -
@JonB & @SimonSchroeder Thanks a lot for your answers let me start of with the threads, I'll only show those where something new shows up:
Now if I click on more in thread 1 (see the previous reply) it doesn't do anything now if I click on it a couple of times, QT crashes without error or any warning.@SimonSchroeder do you mean with this limited amount of stack space that I can't make more of my clickable labels than lets say than a grid filled with them more than (80 on 80) so about 6400?
-
@JonB & @SimonSchroeder Thanks a lot for your answers let me start of with the threads, I'll only show those where something new shows up:
Now if I click on more in thread 1 (see the previous reply) it doesn't do anything now if I click on it a couple of times, QT crashes without error or any warning.@SimonSchroeder do you mean with this limited amount of stack space that I can't make more of my clickable labels than lets say than a grid filled with them more than (80 on 80) so about 6400?
It probably wont help to solve your problem, but why do you need 10.000 labels or more in a
QGridLayoutat the same time? Maybe there is a workaround or better/different solution for your use case.
->QGraphicsView+QGraphicsItemsinstead of labels in a grid. -
Well, you are probably right that I don't need them but for a beginner like me it seemed a logical choice to use labels or buttons since those are things I know. I'm trying to make a forest fire simulation and each tile can be made of a different material that can be changed throughout the simulation. Hence they need to be clickable. I would be open to any suggestions on how to do this better of course!
-
Hi,
As suggested by @Pl45m4 the Graphics View Framework. You'll be able to build your simulation in an easier way to manage your various tiles on the map.
-
Well, you are probably right that I don't need them but for a beginner like me it seemed a logical choice to use labels or buttons since those are things I know. I'm trying to make a forest fire simulation and each tile can be made of a different material that can be changed throughout the simulation. Hence they need to be clickable. I would be open to any suggestions on how to do this better of course!
-
Well, you are probably right that I don't need them but for a beginner like me it seemed a logical choice to use labels or buttons since those are things I know. I'm trying to make a forest fire simulation and each tile can be made of a different material that can be changed throughout the simulation. Hence they need to be clickable. I would be open to any suggestions on how to do this better of course!
So your "map" looks like raster graphics with a low resolution where each label in some color represents a tile of a different material?
Yes, you can do this with the GraphicsView framework.
Of course they can be clickable. But there is so much more you can do, when you use the GraphicsView framework (zooming, custom items etc).
It probably takes some time to re-design your interface but 6.400 or over 10k labels on a widget is by far not the best solution :) -
@Davay one more observations..How many threads do your application has?
in stack trace it's shown upto 21.
