From fe1b8be86b65346724f8f78ab9e5d897e0c480b0 Mon Sep 17 00:00:00 2001
From: Frederick Muriuki Muriithi
Date: Mon, 3 Oct 2022 09:41:52 +0300
Subject: Bug: Do not process data for output too early

* gn3/computations/rust_correlation.py: (parse_tissue_corr_data): The function
  was parsing data from a file, and then converting the parsed values into a
  comma-separated string for output, despite the fact that the data parsed is
  not output in this specific function. This is a really bad idea, and leads
  to errors showing up down the line, far removed from the cause.
---
 gn3/computations/rust_correlation.py | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'gn3')

diff --git a/gn3/computations/rust_correlation.py b/gn3/computations/rust_correlation.py
index 8c3a0d3..03af997 100644
--- a/gn3/computations/rust_correlation.py
+++ b/gn3/computations/rust_correlation.py
@@ -157,9 +157,7 @@ def parse_tissue_corr_data(symbol_name: str,
                 corr_vals = dataset_vals.get(symbol.lower())
 
                 if corr_vals:
-                    corr_vals = [str(trait)] + corr_vals
-
-                    data.append(",".join([str(x) for x in corr_vals]))
+                    data.append([str(trait)] + corr_vals)
 
             except AttributeError:
                 pass
-- 
cgit v1.2.3