blob: 8898bcf7a032d45e3d8ccdfb3ca08c41c8e4abf2 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
|
# Improve Menu Generation and Move it to GN3
## Tags
* assigned: fredm
* type: bug
* priority: medium
* status: in progress
* keywords: menu generation, GN2, GN3
## Description
The
=> https://github.com/genenetwork/genenetwork2/blob/testing/wqflask/wqflask/api/gen_menu.py#L6-L17 gen_dropdown_json
function calls functions that make queries to the database within loops.
This ruins the performance of the system significantly.
The queries should be reworked, and the code should be moved to GN3 since it does database access.
### TODOs
* [ ] Rework the queries: don't run queries inside loops
* [ ] Move the code to GN3
* [x] Provide results of function at an endpoint in GN3
## Update: 2022-05-26
Commits
=> https://github.com/genenetwork/genenetwork3/commit/236d9236c794d7870258eab9e087f990c557462a
=> https://github.com/genenetwork/genenetwork2/commit/e71d2c5d35771645a85a5e0fa7ebdfca4809a16b
=> https://github.com/genenetwork/genenetwork2/commit/98367087f262c0d0cdd7aae691989f90f4d40dcc
enable the fetching of the menu items from GN3. They also report in case of any errors fetching the data from the server.
There is still a need to remove the code from GN2 but it might be a while since it is seemingly used in more than one place.
|
