Open Hrovatin opened 11 months ago
Try some practical examples:
Hi @Hrovatin, this is a very interesting development. As an aside, is there a reason the system
parameter couldn't, instead, be passed as a categorical_covariate_keys
parameter or a continuous_covariate_keys
parameter? Each of these parameters seem to correct for covariates.
We separate the system from the other covariates as we use system for cycle consistency loss (see our preprint). Of course, if you disable the cycle consistency loss, it will boil down to normal cVAE covariate (as are now categorical_covariate_keys
and continuous_covariate_keys
).
If you are interested in using our model, I would suggest using the SysVI implementation that is a PR in scvi-tools (until it becomes official in the next release), as this repository will be only used for reproducibility purposes. The SysVI also enables the integration of multiple systems, as discussed in this issue.
We separate the system from the other covariates as we use system for cycle consistency loss (see our preprint). Of course, if you disable the cycle consistency loss, it will boil down to normal cVAE covariate (as are now
categorical_covariate_keys
andcontinuous_covariate_keys
). If you are interested in using our model, I would suggest using the SysVI implementation that is a PR in scvi-tools (until it becomes official in the next release), as this repository will be only used for reproducibility purposes. The SysVI also enables the integration of multiple systems, as discussed in this issue.
Good to know, thank you for the info!