search

rwdavies / STITCH

STITCH - Sequencing To Imputation Through Constructing Haplotypes
http://www.nature.com/ng/journal/v48/n8/abs/ng.3594.html
GNU General Public License v3.0
74 stars 19 forks source link

Update STITCH.R to add sessionInfo() #39

Closed jelber2 closed 3 years ago

jelber2 commented 3 years ago

Might be useful for some people otherwise ignore.

rwdavies commented 3 years ago

If I do something like this, I think I would do it from inside the STITCH() command, for consistency between command line and R version running. Is there any particular feature of the sessionInfo() you're particularly interested, or the whole thing?

jelber2 commented 3 years ago

I guess I just wanted the whole thing just in case I need to report package versions or something like that.

jelber2 commented 3 years ago

Should I just close the pull request? Like I said, feel free to ignore it.

rwdavies commented 3 years ago

Yeah, I think so. I might come back to it at a later point. STITCH has very few dependencies, and what it uses from those are pretty stable, so those are much less likely to be a source of bugs compared to problems within STITCH, so it is probably fine to just have the user know what version they are running and what command they used while running STITCH