grst
commented
1 week ago Frankly, if the current version works, I don't see a reason to change it. I just thought if it needs to be fixed, it could be easier to rewrite in a more readable language.
Closed nschcolnicov closed 1 week ago
grst
commented
1 week ago Frankly, if the current version works, I don't see a reason to change it. I just thought if it needs to be fixed, it could be easier to rewrite in a more readable language.
pinin4fjords
commented
1 week ago Yep, if it ain't broke don't fix it.
Description of feature
Current version is a combination of bash commands, it could be worth to update it into python or R so that it is easier to read and review by anyone. As discussed in https://github.com/nf-core/differentialabundance/issues/249#issuecomment-2462134567 and https://github.com/nf-core/differentialabundance/issues/249#issuecomment-2476431213.
@grst @pinin4fjords