Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-19 Thread Davide DB
On 17 May 2017 at 21:56, Davide DB wrote: > I uploaded a zip containing 54 eccr dives obtained via dctool from a > JJ Petrel controller. > > https://drive.google.com/open?id=0B1RlIJJRLXsHX3JrcXllUXhDTmM > > All dives use 0,7 and 1,2 as setpoints. Another batch of 24 eccr dives coming from a diff

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-17 Thread Davide DB
I uploaded a zip containing 54 eccr dives obtained via dctool from a JJ Petrel controller. https://drive.google.com/open?id=0B1RlIJJRLXsHX3JrcXllUXhDTmM All dives use 0,7 and 1,2 as setpoints. I hope they could be useful for testing this and future features. Just keep them for future reference.

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Jef Driesen
On 11-05-17 14:37, Anton Lundin wrote: On 11 May, 2017 - Jef Driesen wrote: I replaced your patch with your latest version, and updated the rest of the series. Please have a look. If you are okay with the changes, I'll merge them to master. LGTM. You can add my Reviewed-by: Anton Lundin tag

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Davide DB
On 11 May 2017 at 16:44, Jef Driesen wrote: > > There is no need to send me the other dives. I already managed to extract > them from the log file. I just need to know which dives I should take look The most recent 5 dives are all mCCR with a setpoint of 0,7 Then jump one dive and the 7th is a f

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Jef Driesen
On 2017-05-11 14:20, Davide DB wrote: Ok so I made a mistake. I thought 21 was most recent. I even look carefully at the log trying to see if there was some reference. You can use dctool to parse those raw dive files: ./dctool -f petrel parse -o dive.X.xml dive.X.bin And then you can easily c

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Anton Lundin
On 11 May, 2017 - Jef Driesen wrote: > On 2017-05-10 15:43, Anton Lundin wrote: > >>>That said, I think setpoint values are still interesting to see, to > >>>validate how well the controller did manage to try to keep the o2 close > >>>to the setpoint. > >> > >>After looking at the data, I had the

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Davide DB
Ok so I made a mistake. I thought 21 was most recent. I even look carefully at the log trying to see if there was some reference. I'm out at the office now. I will send you everything again later in the afternoon off list. The dives you have now are a mess because they are logs of early training di

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Jef Driesen
On 2017-05-10 22:16, Davide DB wrote: I downloaded dives from my SF2 Petrel controller via dctool. I attached some of them here. Dives: 17, 18, 19, 20, 21 are mCCR with a setpoint of 0,7 Dive 15 is eCCR with setpoints of 0,7 and 1,3. It was a training dive so you will find OC parts (bailout mode

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-11 Thread Jef Driesen
On 2017-05-10 15:43, Anton Lundin wrote: >That said, I think setpoint values are still interesting to see, to >validate how well the controller did manage to try to keep the o2 close >to the setpoint. After looking at the data, I had the impression that the setpoint value is "unused" because it

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Joakim Bygdell
10 May 2017 at 15:47, Anton Lundin wrote: > On 10 May, 2017 - Joakim Bygdell wrote: > > > On 10 May 2017 at 11:33, Anton Lundin wrote: > > > > > Do we have any data from a Shearwater running as solenoid controller? > > > > I have data from a JJ in eCCr mode. > > There is no sensor data in that

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Davide DB
On 10 May 2017 at 15:47, Anton Lundin wrote: > > Anyway, looking at the data would be interesting, trying to figure out > if there is any bits there disclosing that the computer was running in a > controller mode, and if there are any bits there saying when it fired > the solenoid. I guess you ne

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Steve
Original message From: Jef Driesen Date:10/05/2017 21:37 (GMT+09:30) To: Anton Lundin Cc: Subsurface Subject: Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives On 2017-05-10 11:33, Anton Lundin wrote: > On 10 May, 2017 - Jef Driesen wrote: >

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Anton Lundin
On 10 May, 2017 - Joakim Bygdell wrote: > On 10 May 2017 at 11:33, Anton Lundin wrote: > > > Do we have any data from a Shearwater running as solenoid controller? > > I have data from a JJ in eCCr mode. > There is no sensor data in that log, only setpoint information at the > start, > when the se

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Anton Lundin
On 10 May, 2017 - Jef Driesen wrote: > On 2017-05-10 11:33, Anton Lundin wrote: > >On 10 May, 2017 - Jef Driesen wrote: > >>I've implemented the scaling factor now. See attached set of > >>patches. It took me a bit longer than expected because there were a > >>few cases where the ppO2 still ended

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Davide DB
On 10 May 2017 at 14:33, Joakim Bygdell wrote: > > I have seen similar things on data from mCCRs, fixed setpoint and variable > sensor data, the setpoint of 0.7 seems to be some sort of default value on > shearwaters DCs. > When the DC is used only as a visual reference of the measured ppO2 the >

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Joakim Bygdell
On 10 May 2017 at 14:07, Jef Driesen wrote: > On 2017-05-10 11:33, Anton Lundin wrote: > >> On 10 May, 2017 - Jef Driesen wrote: >> >>> I've implemented the scaling factor now. See attached set of >>> patches. It took me a bit longer than expected because there were a >>> few cases where the ppO2

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Jef Driesen
On 2017-05-10 11:33, Anton Lundin wrote: On 10 May, 2017 - Jef Driesen wrote: I've implemented the scaling factor now. See attached set of patches. It took me a bit longer than expected because there were a few cases where the ppO2 still ended up being zero. And that turned out to be for dives w

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Joakim Bygdell
On 10 May 2017 at 11:33, Anton Lundin wrote: > On 10 May, 2017 - Jef Driesen wrote: > Do we have any data from a Shearwater running as solenoid controller? > > Ex. On the JJ-CCR's they use a Shearwater Petrel with sensors which > controls the o2 solenoid to try to get the o2 sensor values to

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Anton Lundin
On 10 May, 2017 - Anton Lundin wrote: > Attached is the last iteration I did of the sensors patch. It has some > minor differences from the one you included, but most of them isn't that > relevant. > > > The only relevant information is that the adc offset value is probably > in 0.025 mV as unit

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Anton Lundin
On 10 May, 2017 - Jef Driesen wrote: > On 2017-04-17 21:49, Jef Driesen wrote: > >On 14-04-17 22:04, Anton Lundin wrote: > >>On 14 April, 2017 - Jef Driesen wrote: > >> > >>>On 2017-04-13 17:04, Jef Driesen wrote: > >>>I tried a different approach yesterday. Instead of adding 1024 to > >>>the cali

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-05-10 Thread Jef Driesen
On 2017-04-17 21:49, Jef Driesen wrote: On 14-04-17 22:04, Anton Lundin wrote: On 14 April, 2017 - Jef Driesen wrote: On 2017-04-13 17:04, Jef Driesen wrote: I tried a different approach yesterday. Instead of adding 1024 to the calibration value, I simply used the stored value as is, and calcu

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-28 Thread Jef Driesen
On 2017-04-17 22:34, Anton Lundin wrote: On 17 April, 2017 - Jef Driesen wrote: On 14-04-17 21:57, Anton Lundin wrote: >On 13 April, 2017 - Jef Driesen wrote: > >>On 2017-04-12 10:47, Anton Lundin wrote: >>> >>>The issue with current libdivecomputer DC_SAMPLE_PPO2 is that you cant >>>distinguis

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-17 Thread Anton Lundin
On 17 April, 2017 - Jef Driesen wrote: > On 14-04-17 21:57, Anton Lundin wrote: > >On 13 April, 2017 - Jef Driesen wrote: > > > >>On 2017-04-12 10:47, Anton Lundin wrote: > >>> > >>>The issue with current libdivecomputer DC_SAMPLE_PPO2 is that you cant > >>>distinguish between ha "real" "voted" pO

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-17 Thread Jef Driesen
On 14-04-17 22:04, Anton Lundin wrote: On 14 April, 2017 - Jef Driesen wrote: On 2017-04-13 17:04, Jef Driesen wrote: I tried a different approach yesterday. Instead of adding 1024 to the calibration value, I simply used the stored value as is, and calculated the average ppO2 over all three sen

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-17 Thread Jef Driesen
On 14-04-17 21:57, Anton Lundin wrote: On 13 April, 2017 - Jef Driesen wrote: On 2017-04-12 10:47, Anton Lundin wrote: The issue with current libdivecomputer DC_SAMPLE_PPO2 is that you cant distinguish between ha "real" "voted" pO2 and the raw sensor value. I would like to see a option to ex

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-14 Thread Anton Lundin
On 14 April, 2017 - Jef Driesen wrote: > On 2017-04-13 17:04, Jef Driesen wrote: > I tried a different approach yesterday. Instead of adding 1024 to > the calibration value, I simply used the stored value as is, and > calculated the average ppO2 over all three sensors. Then I plotted > all those v

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-14 Thread Anton Lundin
On 13 April, 2017 - Jef Driesen wrote: > On 2017-04-12 10:47, Anton Lundin wrote: > > > >The issue with current libdivecomputer DC_SAMPLE_PPO2 is that you cant > >distinguish between ha "real" "voted" pO2 and the raw sensor value. > > > >I would like to see a option to export both, and be able to

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-14 Thread Jef Driesen
On 2017-04-13 17:04, Jef Driesen wrote: I don't entirely agree with that. If you check the available memory dumps, 5 of them contain CCR dives. I analyzed all of them, and this is what I found: * petrel.marklee.bin The three mV values in the samples are always zero. But since all three O2 sens

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-13 Thread Jef Driesen
On 2017-04-12 10:47, Anton Lundin wrote: On 12 April, 2017 - Davide DB wrote: On 11 April 2017 at 23:44, Jef Driesen wrote: > I'll need to look again at the old discussion, but if I remember > correctly, your patch produced bogus results for some of the data I tested > against. And that's wh

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-12 Thread Jef Driesen
For the Shearwater Petrel CCR divers reading this: Can you supply us with some more data to test against? The Petrel protocol doesn't support memory dumps, so you'll need to dump the individual dives. You can do that with libdivecomputer' dctool: http://www.libdivecomputer.org/builds/stable/linux

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-12 Thread Anton Lundin
On 12 April, 2017 - Davide DB wrote: > On 11 April 2017 at 23:44, Jef Driesen wrote: > > > > > I'll need to look again at the old discussion, but if I remember > > correctly, your patch produced bogus results for some of the data I tested > > against. And that's why I wanted to double check with

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-12 Thread Davide DB
On 11 April 2017 at 23:44, Jef Driesen wrote: > > I'll need to look again at the old discussion, but if I remember > correctly, your patch produced bogus results for some of the data I tested > against. And that's why I wanted to double check with Shearwater. But so > far I wasn't able to get any

RE: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-11 Thread Steve
On 2017-04-09 14:00, Anton Lundin wrote: > On 09 April, 2017 - Joakim Bygdell wrote: >> Jef, how is on the backed side, how many sensors does libdivecomputer >> supports for the Petrel series? > > I sort if figured out now to extract po2 valeues for all sensors from > Shearwater comptuers, but

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-11 Thread Jef Driesen
On 2017-04-09 14:00, Anton Lundin wrote: On 09 April, 2017 - Joakim Bygdell wrote: Jef, how is on the backed side, how many sensors does libdivecomputer supports for the Petrel series? I sort if figured out now to extract po2 valeues for all sensors from Shearwater comptuers, but due to it bei

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Davide DB
On 9 April 2017 at 14:49, Anton Lundin wrote: > I thought about teaching libdivecompter and subsurface to extract, stor > and graph mV values, but last time that came up the discussion ended in > most people would like to have po2 values instead of mV values. Plotting separate sensor mv values w

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Anton Lundin
On 09 April, 2017 - Davide DB wrote: > From what I've been told even Shearwater desktop shows po2 as asingle value > but you can see single sensor values as raw millivolt values. > Given that all the sensor rocket science out there consider mv/po2 being > linear those values are proportional to p

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Davide DB
>From what I've been told even Shearwater desktop shows po2 as asingle value but you can see single sensor values as raw millivolt values. Given that all the sensor rocket science out there consider mv/po2 being linear those values are proportional to po2. Anton I have two ext petrel a CCR here.

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Anton Lundin
On 09 April, 2017 - Willem Ferguson wrote: > Currently Subsurface only explicitly accommodates the Poseidon MKVI > and the APD Inspiration family of rebreathers, having two/three > oxygen sensors by default. The reason is simply: these are the > equipment mostly used by local divers here. Since re

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Anton Lundin
On 09 April, 2017 - Joakim Bygdell wrote: > 9 April 2017 at 11:39, Joakim Bygdell wrote: > > > 9 April 2017 at 11:30, Davide DB wrote: > > > >> On 9 April 2017 at 11:22, Joakim Bygdell wrote: > >> > 9 April 2017 at 11:01, Davide DB wrote: > >> >> > >> >> I update my settings and nothing ha

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Davide DB
Hi Willem, Actually I downoaded the Sweetwater directly via BT with subsurface. I did not install Shearwater we at all. Data format conversion should be libdivecom related or subsurface. ___ subsurface mailing list subsurface@subsurface-divelog.org http:

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Willem Ferguson
Currently Subsurface only explicitly accommodates the Poseidon MKVI and the APD Inspiration family of rebreathers, having two/three oxygen sensors by default. The reason is simply: these are the equipment mostly used by local divers here. Since rebreathers often download only one dive at a time

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Joakim Bygdell
9 April 2017 at 11:39, Joakim Bygdell wrote: > 9 April 2017 at 11:30, Davide DB wrote: > >> On 9 April 2017 at 11:22, Joakim Bygdell wrote: >> > 9 April 2017 at 11:01, Davide DB wrote: >> >> >> >> I update my settings and nothing happens. >> >> I have a Petrel 2 connected to a SF2 CCR with

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Joakim Bygdell
9 April 2017 at 11:30, Davide DB wrote: > On 9 April 2017 at 11:22, Joakim Bygdell wrote: > > 9 April 2017 at 11:01, Davide DB wrote: > >> > >> I update my settings and nothing happens. > >> I have a Petrel 2 connected to a SF2 CCR with three sensors. I get > >> only one green line. > > > > >

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Davide DB
On 9 April 2017 at 11:22, Joakim Bygdell wrote: > 9 April 2017 at 11:01, Davide DB wrote: >> >> > >> > Il 08 apr 2017 13:10, "Joakim Bygdell" ha scritto: >> >> >> >> >> >> Under Preferences -> Graph >> >> You have "Show individual O2 sensors when viewing pO2" >> >> >> >> Once you toggle the "sh

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Joakim Bygdell
9 April 2017 at 11:01, Davide DB wrote: > > > > Il 08 apr 2017 13:10, "Joakim Bygdell" ha scritto: > >> > >> > >> Under Preferences -> Graph > >> You have "Show individual O2 sensors when viewing pO2" > >> > >> Once you toggle the "show pO2" button on in the profile it should show > >> data fro

RE: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Steve
> > Il 08 apr 2017 13:10, "Joakim Bygdell" ha scritto: >> >> >> Under Preferences -> Graph >> You have "Show individual O2 sensors when viewing pO2" >> >> Once you toggle the "show pO2" button on in the profile it should >> show data from all sensors. >> Assuming of course that your divecomputer

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-09 Thread Davide DB
> > Il 08 apr 2017 13:10, "Joakim Bygdell" ha scritto: >> >> >> Under Preferences -> Graph >> You have "Show individual O2 sensors when viewing pO2" >> >> Once you toggle the "show pO2" button on in the profile it should show >> data from all sensors. >> Assuming of course that your divecomputer a

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-08 Thread Davide DB
Ouch! Thanks a lot Il 08 apr 2017 13:10, "Joakim Bygdell" ha scritto: > On 8 April 2017 at 12:35, Davide DB wrote: > >> On 12 January 2015 at 09:41, Willem Ferguson >> wrote: >> >> Sorry guys, really a bad day :) So I bring bag to life this old hot topic. >> >> As far I see there's no way to

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-08 Thread Joakim Bygdell
On 8 April 2017 at 12:35, Davide DB wrote: > On 12 January 2015 at 09:41, Willem Ferguson > wrote: > > Sorry guys, really a bad day :) So I bring bag to life this old hot topic. > > As far I see there's no way to see individual 02 sensor value plots or > am I missing some setting? > Under Prefe

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2017-04-08 Thread Davide DB
On 12 January 2015 at 09:41, Willem Ferguson wrote: > > On 10/01/2015 13:00, Anton Lundin wrote: >> >> >> I've never suggested to remove all the code and start over. I'm >> suggesting better goals to strive towards. >> >> We have discussed o2 sensor plotting on this mailinglist for 2-3 years, >> a

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-12 Thread Willem Ferguson
On 10/01/2015 13:00, Anton Lundin wrote: I've never suggested to remove all the code and start over. I'm suggesting better goals to strive towards. We have discussed o2 sensor plotting on this mailinglist for 2-3 years, and everytime before we have come to the conclusion that plotting individua

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-10 Thread Anton Lundin
On 09 January, 2015 - Willem Ferguson wrote: > On 09/01/2015 16:32, Anton Lundin wrote: > > > >I've bin thinking if this is a good idea to plot in our already massive > >plot. > > > >I'm more leaning towards just plotting a computed po2 value here, and > >the setpoint of course, and then rather ad

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-09 Thread Willem Ferguson
On 09/01/2015 16:32, Anton Lundin wrote: I've bin thinking if this is a good idea to plot in our already massive plot. I'm more leaning towards just plotting a computed po2 value here, and the setpoint of course, and then rather add a some sort of separate tool to analyze the o2 cells, and how

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-09 Thread Anton Lundin
On 09 January, 2015 - Willem Ferguson wrote: > This patch implements the visualisation of the oxygen sensor data for CCR > dives. The partial pressure measurements of each of the sensors are graphed > using the existing partial pressure axis in the dive profile widget. There > are now four separat

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-09 Thread Willem Ferguson
On 09/01/2015 11:20, Davide DB wrote: Any way to display OX sensor values (if available) in a PSCR dive? I have no experience whatsoever with PSCR. Can these data be logged into the existing fields of the structures of sample as defined in dive.h? I suspect that the most important impedimen

Re: [PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-09 Thread Davide DB
Any way to display OX sensor values (if available) in a PSCR dive? ___ subsurface mailing list subsurface@subsurface-divelog.org http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

[PATCH] Visualisation of individual oxygen sensor data for CCR dives

2015-01-09 Thread Willem Ferguson
This patch implements the visualisation of the oxygen sensor data for CCR dives. The partial pressure measurements of each of the sensors are graphed using the existing partial pressure axis in the dive profile widget. There are now four separate sets of data that can be plotted using the partial