From 77e7ea7b7fb9fd2961526122a41512fd00da9858 Mon Sep 17 00:00:00 2001
From: Frederick Muriuki Muriithi
Date: Mon, 24 Oct 2022 03:19:11 +0300
Subject: Issues (decouple-tasks-in-results-display): New issue

* issues/decouple-tasks-in-results-display.gmi: add a new issue.
---
 issues/decouple-tasks-in-results-display.gmi | 13 +++++++++++++
 1 file changed, 13 insertions(+)
 create mode 100644 issues/decouple-tasks-in-results-display.gmi

(limited to 'issues/decouple-tasks-in-results-display.gmi')

diff --git a/issues/decouple-tasks-in-results-display.gmi b/issues/decouple-tasks-in-results-display.gmi
new file mode 100644
index 0000000..41fa057
--- /dev/null
+++ b/issues/decouple-tasks-in-results-display.gmi
@@ -0,0 +1,13 @@
+# Decouple Tasks in Correlations Results Display
+
+## Tags
+
+## Description
+
+=> https://github.com/genenetwork/genenetwork2/blob/f03127d751cd629596c2595d700fbb044725e01c/wqflask/wqflask/correlation/show_corr_results.py#L112 The loop processing the results
+is doing way too much.
+
+We need to slowly decouple the different tasks into separate functions that can be called successively on each individual trait to build up the results, with something like the `map` function, or list comprehensions.
+
+=> https://github.com/genenetwork/genenetwork2/blob/f03127d751cd629596c2595d700fbb044725e01c/wqflask/wqflask/correlation/show_corr_results.py#L116-L122 The conditional fetching of traits data
+should probably be done before we enter the looping part of the results processing to make the logic easier and decouple the fetching from the results processing.
-- 
cgit v1.2.3