You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As far as I can tell, changing the sex option in the config.yaml file does not actually change anything about the calculation. The --gender option does not appear in the shell command for runTitanCNA rule, so titanCNA.R will always use the default gender which is male. Was this intentional? Is there some reason we shouldn't be specifying the gender?
The text was updated successfully, but these errors were encountered:
You're correct - in the snakemake pipeline the gender/sex options are not currently being used and defaults to male which excludes chrX.
I was in the middle of adding the feature to predict the gender from the ichorCNA analysis so that users can use None if the cohort includes both males and females.
For now, I will enable the user to use male or female but this will not work for cohorts with both.
For now, assumes that sample set contains all males or all females but not both. #72
For sample sets with both, please use `male` so that chrX is excluded allelic CN analysis
The gender field could be optional and default to male if not present so as not to interrupt workflows based on older versions. I can make up a pull request for this.
An additional rule is added to TitanCNA.snakefile to combine the results of TITAN with ichorCNA. In particular, chrX results from ichorCNA is include for male samples because TitanCNA does not include chrX for males.
Addresses issues #53
To some extent is related to #72
As far as I can tell, changing the sex option in the config.yaml file does not actually change anything about the calculation. The --gender option does not appear in the shell command for runTitanCNA rule, so titanCNA.R will always use the default gender which is male. Was this intentional? Is there some reason we shouldn't be specifying the gender?
The text was updated successfully, but these errors were encountered: