FET Forum

FET Development => Report a Bug => Topic started by: liquid on July 18, 2018, 08:34:53 PM

Title: assertion failed
Post by: liquid on July 18, 2018, 08:34:53 PM
For the first time, I met the following assertion failure. It does not mean anything for me. What about you?

fet: interface/timetableviewstudentstimehorizontalform.cpp:540: void TimetableViewStudentsTimeHorizontalForm::updateStudentsTimetableTable(): Assertion `d*gt.rules.nHoursPerDay+h<studentsTimetableTable->columnCount()' failed.
Aborted


It happened while closing the dialog after finished successful generation.
Title: Re: assertion failed
Post by: Volker Dirr on July 18, 2018, 08:40:08 PM
that should never happen :-(
What version do you use?
Can you please sent me or Liviu your dataset? In best case the the data_and_timetable.fet file from the fet results folder. we will keep it private, but that will hepl us to understand the bug.
Title: Re: assertion failed
Post by: Volker Dirr on July 18, 2018, 08:41:32 PM
BTW: did you modify the number of hours per day or the number of days per week after generating?
Title: Re: assertion failed
Post by: Liviu Lalescu on July 18, 2018, 09:29:42 PM
I cannot replicate. I modified the number of days and hours. It is stable.

Please try to replicate. Send us your file and tell us the FET version you use.
Title: Re: assertion failed
Post by: liquid on July 18, 2018, 10:00:32 PM
Quote from: Volker Dirr on July 18, 2018, 08:41:32 PM
BTW: did you modify the number of hours per day or the number of days per week after generating?
Yes, I did.
If I replicate this I'll let you know.
Title: Re: assertion failed
Post by: Volker Dirr on July 19, 2018, 07:51:06 AM
What version did you use?
I tried a lot, but i can't reproduce.
Title: Re: assertion failed
Post by: Liviu Lalescu on July 19, 2018, 07:52:48 AM
I can  :-[

Generate, view timetable (don't close this view until the crash), increase number of hours or days, generate again.

I will fix this, but I am not sure when I'll release, because I just released 2 days ago.
Title: Re: assertion failed
Post by: Volker Dirr on July 19, 2018, 08:01:31 AM
Ah... Don't close the view is the trick.
I fear we need to recheck all modeless dialogs.
Title: Re: assertion failed
Post by: Liviu Lalescu on July 19, 2018, 04:27:12 PM
Thank you for the bug report!

I fixed this bug. I hope you will accept my solution, which is not perfect. Please see the Snapshots announcement and let me know. I intend to release in 1 or more days.