On Mon, Jul 21, 2014 at 8:16 AM, Gehad Elrobey <gehadelro...@gmail.com> wrote:
> > Hi miika, > > On Sun, Jul 20, 2014 at 8:02 AM, Miika Turkia <miika.tur...@gmail.com> > wrote: > >> New bugs I spotted: >> >> - The tooltip on temperature graphs shows actually the time (concatenated >> with temperature unit). >> > > what is it supposed to show? I think this is consistent will all the other > curves in the profile. > It should show the temperature value. The temperature graph is drawn based on the temperature correctly, but the tooltip shows time since start of the time, not the temperature. > I Also fixed the previous patches, and attached some new patches with them. > There are still three bugs that I am working on them now. > Thanks, I'll have a look later. > - I am not sure about the search of location - e.g. if I write "Mexico >> somethingthatdoesnotexist", I get all the locations in Mexico. >> - Zooming in (ctrl++) and out (ctrl+-) and restoring zoom level (ctrl+0) >> on firefox does not rescale the graph properly >> - The heading and gaschange switches should use the value field from the >> event, not the type field (type is still internal to subsurface, and does >> not provide any useful information to the users, on the other hand value >> would e.g. tell what gas we changed to) >> > > Regarding the event value, is exporting the event value as a number is > useful? or there is some enum strings that must be shown instead? > The heading is degrees, as in compass. The gas change event's value contains the o2 per centage, of the gas mix, or it can also contain the o2 and he content. Low order bits being o2 and high order bits the he: he = cur_event.value >> 16 o2 = cur_event.value & 0xffff miika > - If I go to next dive on detailed view, and hit one extra right arrows >> after reaching the last dive, first click on left does not change the >> selection, only re-clicking on left does >> >> I'll still do a bit of poking around once I have more time... >> >> miika >> >> >> On Sat, Jul 19, 2014 at 6:30 AM, Gehad Elrobey <gehadelro...@gmail.com> >> wrote: >> >>> I Attached some bug fixes, Still working on the remaining bugs. >>> >>> Regards, >>> Gehad. >>> >>> >>> On Tue, Jul 15, 2014 at 11:22 PM, Tomaz Canabrava <tcanabr...@kde.org> >>> wrote: >>> >>>> On Tue, Jul 15, 2014 at 4:31 PM, Anton Lundin <gla...@acc.umu.se> >>>> wrote: >>>> > On 15 July, 2014 - Anton Lundin wrote: >>>> > >>>> >> On 15 July, 2014 - Miika Turkia wrote: >>>> >> >>>> >> > A few more bugs: >>>> >> > - Trimix dives are missing He content (use e.g. test20.xml for >>>> testing >>>> >> > purposes) - Reported by Roberto, but mailing list was not included >>>> >> >>>> >> Gaah. This is code that uses the event data straight off. In the >>>> case of >>>> >> SAMPLE_EVENT_GASCHANGE especially we need to map the event to the >>>> >> available cylinders and that way figure out which gas we are using. >>>> >> >>>> >> Almost every place where we use get_gasmix_from_event we should be >>>> using >>>> >> get_cylinder_index instead and grab the "real" gasmix from the >>>> cylinder, >>>> >> except the case where get_gasmix_from_event is called from >>>> >> get_cylinder_index =) >>>> >> >>>> >> >>>> >> This was more work than anticipated when i first gave it a stab. If >>>> no >>>> >> one else untangles this I'll give it another stab later. >>>> >> >>>> >> >>>> > >>>> > Gaah 2. >>>> > >>>> > I of course miss-read the report and i was talking about what we write >>>> > in the regular-non-html-ui. >>>> > >>>> > I found another bug. >>>> >>>> :P >>>> >>>> > //Anton >>>> _______________________________________________ >>>> subsurface mailing list >>>> subsurface@hohndel.org >>>> http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface >>>> >>> >>> >> >
_______________________________________________ subsurface mailing list subsurface@hohndel.org http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
