moscot
moscot copied to clipboard
SinkhornProblem star policy wrong problem order
sp = SinkhornProblem(adata_red)
sp = sp.prepare(
key="perturbation", joint_attr="X_pca", policy="star", reference="control"
)
sp = sp.solve(1e-2, 0.95, 0.95)
sp
generates SinkhornProblem[('Dacinostat (LAQ824)', 'control'), ('Givinostat (ITF2357)', 'control'), ('TAK-901', 'control'), ('Flavopiridol HCl', 'control')]
but should be SinkhornProblem[('control', 'Dacinostat (LAQ824)'), etc ]
@MUCDK why is this an issue? reference is always the target in star policy https://github.com/theislab/moscot/blob/ca135725193b70541f054d8cd853b0dee76f3bb0/src/moscot/utils/subset_policy.py#L312
should we make it more flexible then? Here, the control should always be the source.
I'd be ok with that, I just don't super see how this would change/unblock the perturbation setting. At the end of the day it is just convention right, since everything is symmetric in moscot?
hi, I overlooked this issue from a long time ago. I don't understand if this is a design problem or a feature etc.