From dd0644c5a96d530c930cd5ff43c3cead7d8a87b7 Mon Sep 17 00:00:00 2001 From: Frederick Muriuki Muriithi Date: Fri, 25 Nov 2022 07:34:42 +0300 Subject: issues: decouple-tasks-in-results-display: Add note on bulk querying * issues/decouple-tasks-in-results-display.gmi: Add notes on bulk querying of the metadata. --- issues/decouple-tasks-in-results-display.gmi | 11 +++++++++++ 1 file changed, 11 insertions(+) (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 index af4ee1c..ab1ccd5 100644 --- a/issues/decouple-tasks-in-results-display.gmi +++ b/issues/decouple-tasks-in-results-display.gmi @@ -25,3 +25,14 @@ should probably be done before we enter the looping part of the results processi pr to fix this issue => https://github.com/genenetwork/genenetwork2/pull/742 + +## 2022-11-25 + +### Bulk Querying + +The code in the +=> https://github.com/genenetwork/genenetwork2/blob/d318b1d649a3ece1a5a6c24bcd3496e14dbc69cd/wqflask/wqflask/correlation/show_corr_results.py#L169-L179 `generate_table_metadata` function +is still fetching the data one trait at a time, which is a performance bottleneck. + +This could be changed to operate +=> https://github.com/genenetwork/genenetwork3/blob/4a9fd93d01b6d4bd9d9880dbf2274b3c7f2de37b/gn3/db/partial_correlations.py#L751-L791 something like this -- cgit v1.2.3