From e96a1864be076fcdf870188b95b1d43f16308590 Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Tue, 19 Jun 2012 22:41:44 -0700 Subject: [PATCH] Fix cochran CSV pressure data import The cochran CSV pressure data is actually in units of '4 psi', not in just psi. That seems to be the resolution cochran internally keeps things in, and unlike the depth reading there's no conversion to standard units in the export (for depth, the quarter-foot depth resolution is converted to tenths of feet when exporting). Yeah, none of this makes any sense to me either, but I knew it was the case. I had just forgotten that factor-of-four when I did the importer. With this fix, I get the same subsurface data (modulo some rounding differences particularly for temperature) whether I go through David McNett's UDDF converter, or just import the CSV data directly. Signed-off-by: Linus Torvalds --- file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/file.c b/file.c index 3f06259..538f5c7 100644 --- a/file.c +++ b/file.c @@ -140,7 +140,7 @@ static void add_sample_data(struct sample *sample, enum csv_format type, double sample->temperature.mkelvin = F_to_mkelvin(val); break; case CSV_PRESSURE: - sample->cylinderpressure.mbar = psi_to_mbar(val); + sample->cylinderpressure.mbar = psi_to_mbar(val*4); break; } } -- 2.43.0