Hi Linus,
Am 14.09.2017 um 21:20 schrieb Linus Torvalds: > From: Linus Torvalds <torva...@linux-foundation.org> > Date: Thu, 14 Sep 2017 12:14:33 -0700 > Subject: [PATCH 2/2] Properly clear sensor pressure data for synthetic > plotinfo entries > > We only cleared the first sensor data when we created new synthetic plot > info entries, because we only used to have one (well, we had the o2 > data, but apparently nobody ever noticed that it didn't get properly > interpolated, probably because people who have CCR dives with o2 > pressures are few, and the pressure drops are gradual anyway). > > Clear all the pressure data, so that the interpolation code doesn't > think we have some existing real sensor data for the plot info entries > in between proper sample entries. > > Signed-off-by: Linus Torvalds <torva...@linux-foundation.org> > --- > > This fixes the very odd "sometimes interpolation doesn't work". Sorry for the maybe stupid questions: This does already solve the issue with the spike I reported, isn't it? I first was thinking no, this now is only taking care about s.th. additional you discovered during your investigations. But after reading everything again and testing the patch I now believe this is already the final fix?! What is clear is that the fix only helps for any newly planned dives after applying the fix. Any old dive from the planner with the specific setup will still show the spike because there is the "incorrect" pressure info in the XML. To fix old dives I need to remove the pressure info manually in the XML. Please correct me if I'm wrong! Thanks! Best regards Stefan -- Stefan Fuchs E-Mail: sfu...@gmx.de <mailto:sfu...@gmx.de>
_______________________________________________ subsurface mailing list subsurface@subsurface-divelog.org http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
