From d5ebd53c245667f3d3e3c22e6b8b2fc299b805be Mon Sep 17 00:00:00 2001 From: Robrecht Cannoodt Date: Wed, 29 Jul 2026 14:13:30 +0200 Subject: [PATCH] stop copying train_mod2 into novel's model artifact novel_predict only reads uns from that file -- modality, model_dim and removed_vars -- so there's no reason to carry the expression matrix, obs and var along with it. --- CHANGELOG.md | 6 ++++++ src/methods/novel/novel_train/script.py | 3 ++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e9a8396c..6c95f484 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,9 @@ +# task_predict_modality 0.2.0 + +## MINOR CHANGES + +* `novel_train`: Store only the metadata `novel_predict` needs in the model artifact, instead of a full copy of `train_mod2` (PR #47). + # task_predict_modality 0.1.1 ## NEW FUNCTIONALITY diff --git a/src/methods/novel/novel_train/script.py b/src/methods/novel/novel_train/script.py index 0cd94124..8780723d 100644 --- a/src/methods/novel/novel_train/script.py +++ b/src/methods/novel/novel_train/script.py @@ -38,7 +38,8 @@ input_train_mod1 = ad.read_h5ad(par['input_train_mod1']) input_train_mod2 = ad.read_h5ad(par['input_train_mod2']) -adata = input_train_mod2.copy() +# carries the metadata novel_predict needs; the matrix itself is not used there +adata = ad.AnnData(uns=dict(input_train_mod2.uns)) mod1 = input_train_mod1.uns['modality'] mod2 = input_train_mod2.uns['modality']