saezlab / transcriptutorial

This is a tutorial to guide the analysis of RNAseq dataset using footprint based tools such as DOROTHEA, PROGENY and CARNIVAL
https://saezlab.github.io/transcriptutorial/
GNU General Public License v3.0
55 stars 30 forks source link

Error in sample_resolution_carnival.R obtaining tfList with larger ncol data #12

Closed IvanEllson closed 3 years ago

IvanEllson commented 4 years ago

Hello, thank you for the tutorial. In the _06_analysis_CARNIVALresults tutorial you use a carnival sample resolution object generated from the _sample_resolutioncarnival.R script (I think maybe you should explain in the tutorial where it comes from in order to understand it better)

I think I found an error in the code of this _sample_resolutioncarnival.R script, line 22:

tfList = generateTFList(tf_activities_carnival, top=50, access_idx = 1:nrow(tf_activities_carnival))

where it should be access_idx = 1:ncol(tf_activities_carnival)

I found that error becouse my data set have more samples than TFs and my resulting tfList was giving me a list with missing samples. I hope it helps. Thank you.

rosherbal commented 3 years ago

Dear @IvanEllson,

Thanks for your feedback! I have fixed the reported bug and added a note in the main notebook referring the sample resolution scripts :)

All the best,

Rosa