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
drop/requirementsR.txt specifies installation of OUTRIDER, FRASER, and tMAE to install from the HEAD of their respective git repositories.
This leads to two problems:
The pipeline is not necessarily reproducible if there are changes made to any of those repositories.
The pipeline can unexpectedly break with respect to those changes.
The second problem happened to me this week. FRASER was updated last week to have strand-specific counting per sample (rather than globally set over analyses): gagneurlab/FRASER#72. However, the scripts in the DROP repository have not been updated. This causes an error in aberrant-splicing-pipeline/Counting/01_1_countRNA_splitReads_samplewise.R, line 38; the conditional has length > 1, which throws an exception.
While DROP itself needs to be updated to reflect FRASER's new behavior, I think DROP should also specify fixed commits for installation to ensure that changes to the HEAD of the git repositories on which it is dependent do not break DROP or affect its reproducibility between DROP versions.
The text was updated successfully, but these errors were encountered:
It looks like #537 will hopefully address the immediate issue I ran into with the updated version of FRASER.
I would still hope for some kind of predictable versioning of DROP's dependencies. If one were to publish an analysis using DROP, how should they report the version of OUTRIDER/FRASER/tMAE that they used? How could someone else wanting to reproduce their analysis set up a matching environment that didn't integrate changes made to these dependencies?
Hi Joseph. Thanks again for raising these concerns. With v 1.3.4 we have tried to be more robust to different versions of packages and dependencies. We now install only specific versions. We recommend you install 1.4.0, which further contains improvements to FRASER.
Hi Joseph, as explained above, this should no longer be an issue since we no longer install dependencies from the HEAD of the git repositories as of drop 1.3.4. Let us know if you have any more questions, otherwise I will close the issue in two weeks.
drop/requirementsR.txt
specifies installation of OUTRIDER, FRASER, and tMAE to install from the HEAD of their respective git repositories.This leads to two problems:
The second problem happened to me this week. FRASER was updated last week to have strand-specific counting per sample (rather than globally set over analyses): gagneurlab/FRASER#72. However, the scripts in the DROP repository have not been updated. This causes an error in aberrant-splicing-pipeline/Counting/01_1_countRNA_splitReads_samplewise.R, line 38; the conditional has length > 1, which throws an exception.
While DROP itself needs to be updated to reflect FRASER's new behavior, I think DROP should also specify fixed commits for installation to ensure that changes to the HEAD of the git repositories on which it is dependent do not break DROP or affect its reproducibility between DROP versions.
The text was updated successfully, but these errors were encountered: