Conversation
… signatureSearch to query LINCS) Merge branch 'dea' of github.com:lasseignelab/setbp1_hd into sigsearch
… signatureSearch to query LINCS) Merge branch 'dea' of github.com:lasseignelab/setbp1_hd into sigsearch
…e score signature)
…e score signature)
Conflicts: results/dea/consensus_signature/rra/compositescore_consensus_signature.csv results/dea/consensus_signature/rra/compositescore_full_table_degs.csv
Conflicts: results/dea/consensus_signature/rra/compositescore_consensus_signature.csv results/dea/consensus_signature/rra/compositescore_full_table_degs.csv
|
@tsoelter I updated the cap verification and also had to pull updates from main after the consensus signature was merged, so it dismissed your review |
tsoelter
left a comment
There was a problem hiding this comment.
The updated verifications and all changes work perfectly! 🎉
|
Putting review on hold--need to add "pediatric" drug label excerpts for dailymed query and update runs with new prioritized SETBP1 targets |
…d into sigsearch Conflicts: .gitignore config/environments/lasseignelab.sh # # It looks like you may be committing a merge. # If this is not correct, please remove the file # .git/MERGE_HEAD # and try again. # Please enter the commit message for your changes. Lines starting # with '#' will be kept; you may remove them yourself if you want to. # An empty message aborts the commit. # On branch sigsearch # All conflicts fixed but you are still merging. # (use "git commit" to conclude merge) # # Changes to be committed: # # modified: .gitignore # modified: config/environments/lasseignelab.sh # new file: data/260403_setbp1_targets.csv # new file: results/figures/SETBP1_target_exp_gtex_heatmap.png # new file: results/figures/SETBP1_target_sex_bytissue_boxplot.png # new file: results/figures/SETBP1_temporal_trajectory.png # new file: results/figures/supplementary_figure_1.png # new file: results/permutation_analysis/SETBP1_target_coexpression_brainspan.csv # new file: results/permutation_analysis/brainspan_metadata_dev_windows.csv # new file: results/permutation_analysis/brainspan_metadata_donors_by_age.png # new file: results/permutation_analysis/brainspan_permutation_stats.csv # new file: results/permutation_analysis/brainspan_window_gene_means.csv # new file: results/permutation_analysis/gtex_metadata_donors.png # new file: results/permutation_analysis/gtex_permutation_results.csv # new file: results/permutation_analysis/gtex_sex_wilcox_results.csv # new file: results/permutation_analysis/gtex_target_coexpression.csv # new file: results/permutation_analysis/gtex_target_sex_expression.csv # new file: results/permutation_analysis/prioritized_setbp1_targets.csv # new file: results/permutation_analysis/prioritized_target_coexpression.csv # new file: results/permutation_analysis/prioritized_target_pathways.csv # new file: src/figures/01_supp_figure.sh # new file: src/figures/supp_figure_1.R # new file: src/permutation_analysis/01_fetch_temporal_exp.sh # new file: src/permutation_analysis/02_job_permutation.sh # new file: src/permutation_analysis/02_spatiotemporal_perm.R # new file: src/permutation_analysis/03_job_permutation.sh # new file: src/permutation_analysis/03_postnatal_development.R # new file: src/permutation_analysis/04_job_prioritize.sh # new file: src/permutation_analysis/04_prioritize_targets.R # new file: verifications/permutations_fetchdata.out # new file: verifications/permutations_fetchdata.sh # new file: verifications/permutations_prioritize.out # new file: verifications/permutations_prioritize.sh # new file: verifications/permutations_run.out # new file: verifications/permutations_run.sh #
sashataluri
left a comment
There was a problem hiding this comment.
The following command ls -hal data/dailymed produces this error for me: ls: cannot access data/dailymed: No such file or directory
Not sure why I am getting it. Maybe an issue on my end. Everything works fine otherwise, and the code looks good! I verified the MD5 sums, which were clear. I can approve it when you get a chance to look at this.
tsoelter
left a comment
There was a problem hiding this comment.
Everything still looks good, works, and verifies on my end! 🎉
Hmm, can you check your log files for the 13_job_drug_safety run? (i.e., and let me know what it shows? |
Summary
These scripts include the initial signature reversion analysis:
Author Checklist
See instructions here.
cap_container command has been included in each script using a container.
in this pull request.
or unrelated files are part of this pull request. Use
.gitignoreto preventirrelevant files from accidentally being included.
lasseignelabanddefaultenvironments by running the job until two consecutive runs producethe same results. Code comments (per script) have been added for
non-reproducible scripts. Include information about why the script is not
reproducible.
message should show in the GitHub pull request status box above the
"Merge pull request" button.
branch has conflicts that must be resolved" message in the GitHub pull
request status box above the "Merge pull request" button.
pull request.
Self/Peer Review Checklist (Coding Guidelines)
why, not what the code is doing. Avoid excessive or redundant commenting.
Setup
Make sure you're on cap version 1.0
Reproduce results
These scripts must be ran in order and cannot be run at the same time (each one depends on files in the prior script).
Time approximations do not include pulling docker image and converting to a sif file if it's not already available! This may add 10-20 minutes to the run time.
1) Run the signature reversion script (~10-15 minutes)
2) Run the drug target analysis script (~1-2 minutes)
3) Run script to pull drug safety data from DailyMed API (~10-15 minutes)
STOP
Make sure the drug safety script ran fine. You should see an output, but it will not reproduce.
If everything ran fine, then you can remove your copy and symlink my copy:
data/dailymed should now be symlinked to the copy in PROJECT_dir in the Lasseigne Lab Cheaha directory.
4) Run plotting script for figure 5 ( < 2 minutes)
Verification
Run verification script
If nothing is returned, the outputs match!
Note: The
./results/signature_reversion/drug_safety_data.csvis not reproducible because it's using the DailyMed API which updates daily. The script has a comment noting this, and it is excluded from the md5 verification. The specific version used will be added to Zenodo