about summary refs log tree commit diff
path: root/gn3/db/traits.py
diff options
context:
space:
mode:
authorFrederick Muriuki Muriithi2025-10-22 13:27:54 -0500
committerFrederick Muriuki Muriithi2025-10-22 13:27:54 -0500
commit05013c13710eccadb69746bf186eb672c2af69c4 (patch)
tree71a706fd30bf0d535a0fab737bb0e42a1667e60b /gn3/db/traits.py
parent977efbb54da284fb3e8476f200206d00cb8e64cd (diff)
downloadgenenetwork3-05013c13710eccadb69746bf186eb672c2af69c4.tar.gz
Fetch configs from app setting and pass them down
Fix the code to avoid using global variables holding configuration
variables to avoid the failures caused by action-at-a-distance effects
due to change of a value elsewhere.

This way, all code consistently receives the same configuration values
passed on from callers.

We pass on the following configuration settings:
* TMPDIR
* SQL_URI
* TEXTDIR
Diffstat (limited to 'gn3/db/traits.py')
-rw-r--r--gn3/db/traits.py3
1 files changed, 1 insertions, 2 deletions
diff --git a/gn3/db/traits.py b/gn3/db/traits.py
index fa13fcc..fbac0da 100644
--- a/gn3/db/traits.py
+++ b/gn3/db/traits.py
@@ -3,7 +3,6 @@ import os
 from functools import reduce
 from typing import Any, Dict, Sequence
 
-from gn3.settings import TMPDIR
 from gn3.chancy import random_string
 from gn3.function_helpers import compose
 from gn3.db.datasets import retrieve_trait_dataset
@@ -690,7 +689,7 @@ def retrieve_trait_data(trait: dict, conn: Any, samplelist: Sequence[str] = tupl
     return {}
 
 
-def generate_traits_filename(base_path: str = TMPDIR):
+def generate_traits_filename(base_path: str):
     """Generate a unique filename for use with generated traits files."""
     return (
         f"{os.path.abspath(base_path)}/traits_test_file_{random_string(10)}.txt")