Skip to content

Commit 9aa617f

Browse files
committed
Updated docstrings and comments to remove stale references
1 parent 4ff3be5 commit 9aa617f

File tree

2 files changed

+7
-12
lines changed

2 files changed

+7
-12
lines changed

client/plots/volcano/view/DataPointMouseEvents.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export class DataPointMouseEvents {
2727
})
2828
circle.on('click', async () => {
2929
if (termType === DNA_METHYLATION) {
30-
// For DM, launch GPDM probe-level analysis
30+
// For DM, launch GPDM region-level analysis
3131
const geneName = d.gene_name?.split(',')[0]?.trim()
3232
if (geneName) {
3333
const promoterId = 'promoter_id' in d ? (d as any).promoter_id : undefined

python/src/gpdm_analysis.py

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,11 @@
4646
import numpy as np
4747
import h5py
4848

49-
# Silence GPDM's module-level logger BEFORE importing the gpdm package.
50-
# gpdm/core.py calls logging.basicConfig(level=INFO) at module load time,
51-
# which would direct INFO messages to stderr. run_python() treats any stderr
52-
# output as a fatal error, so we must set the level to CRITICAL before the
53-
# import triggers basicConfig.
49+
# Silence GPDM's logger so no INFO messages reach stderr.
50+
# run_python() treats any stderr output as a fatal error.
5451
logging.getLogger("gpdm").setLevel(logging.CRITICAL)
5552

56-
# Set matplotlib to non-interactive Agg backend before gpdm import,
57-
# since gpdm/core.py imports matplotlib.pyplot at module load time.
53+
# Set matplotlib to non-interactive Agg backend before any plotting imports.
5854
import matplotlib
5955
matplotlib.use('Agg')
6056

@@ -344,8 +340,7 @@ def run_gpdm(params):
344340
except Exception:
345341
pass # non-fatal: analysis result still returned without image
346342

347-
# --- Step 6: Build grid response for the D3 visualization (termdb/gpdm) ---
348-
# termdb/dmr ignores this; termdb/gpdm needs it for all 4 visualization panels.
343+
# --- Step 6: Build grid response for visualization ---
349344
grid_df = analysis.to_dataframe()
350345

351346
def safe_list(arr):
@@ -389,7 +384,7 @@ def safe_list(arr):
389384
return {
390385
'status': 'ok',
391386
'dmrs': annot_dmrs,
392-
'naive_dmrs': [], # naive model not run; kept for termdb/gpdm client compatibility
387+
'naive_dmrs': [], # naive model not run; placeholder for future use
393388
'grid': grid,
394389
'metadata': {
395390
'n_probes': int(len(positions)),
@@ -442,7 +437,7 @@ def default(self, obj):
442437
# - Always exit with code 0 (non-zero exit causes run_python to reject the result)
443438
#
444439
# Errors are communicated as {"error": "message"} in the JSON stdout so the
445-
# Node route handler (termdb.gpdm.ts) can surface them to the client.
440+
# Node route handler (termdb.dmr.ts) can surface them to the client.
446441
try:
447442
# Read the complete stdin payload (run_python pipes the request body here)
448443
input_data = sys.stdin.read()

0 commit comments

Comments
 (0)