Planner Issue

[Tomaz Canabrava]

On Sat, May 31, 2014 at 10:50 AM, Dirk Hohndel <dirk at hohndel.org> wrote:
> On Sat, May 31, 2014 at 06:34:00AM -0700, Dirk Hohndel wrote:
>> On Sat, May 31, 2014 at 03:20:06PM +0200, Joakim Bygdell wrote:
>> > >
>> > > So something goes horribly wrong copying the cylinders. Again, very odd if
>> > > I look at the code.
>> > >
>> > > I can start the planner here just fine with a dive file with 300+ dives
>> > > and a myriad of gas mixes.
>> > >
>> > > I wonder if it's unused cylinders on the existing dive that throw things
>> > > off - I tend to purge those. Let me find a dive where I have those and see
>> > > what happens
>> > >
>> > > Or alternatively, could you send us an XML (as small as possible) with
>> > > which to reproduce the bug? "open this XML, select dive ###, hit planner,
>> > > this bad thing happens"
>> > >
>> > > /D
>> >
>> > See if this is enough.
>>
>> As I assumed, you have unused cylinders in that dive.
>> That said, it happily opens a fully functional planner for me.
>
> And the reason for that is that I ended up in detached head state
> yesterday and ended up not pushing the changes that I had in my local
> tree.
>
> Oops. Sorry about that.
>
> Now this is all merged and things should work better.
>
> I don't think this addresses the ruler/Qt crash that Henrik and Joakim are
> seeing.
>
> Tomaz, could you send a patch to disable the ruler on the planner? I'm
> running out of time this morning.
>
> Thanks

Will do.
I'm working on planner issues today so expect a few patches by the end
of the day.
One thing is that the other patches doesn't fix the issue we got (
same bug with the same fix: do not set the cylinder's depth seems to
fix it, for some reason ) - I'm trying to poke at the planner.c code
that actually creates the dive to see what's going on.

Tomaz

> /D