Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show which STAR binary+version is executed #1157

Merged
merged 1 commit into from
Mar 6, 2021
Merged

Conversation

smoe
Copy link
Contributor

@smoe smoe commented Feb 24, 2021

This is not really a PR since this needs to be adapted to your style and you may be aware of tools parsing that line. It is more an expression of despair on my side. STAR for me is wrapped by pigx-rnaseq. I got a bit into trouble with my local installation and now thought I should better have a look at the official guix-provided installation and am not fully confident that the STAR binary executed is the one I installed or the one that guix installed. The log file looks like

sh: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
/gnu/store/mmhimfwmmidf09jw1plw3aw1g1zn2nkh-bash-static-5.0.16/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
Feb 23 21:54:52 ..... started STAR run
Feb 23 21:54:57 ..... loading genome

EXITING: FATAL INPUT ERROR: unrecognized parameter name "genomeType" in input "genomeParameters.txt"
SOLUTION: use correct parameter name (check the manual)

Feb 23 21:54:57 ...... FATAL ERROR, exiting
/gnu/store/pwcp239kjf7lnj5i4lkdzcfcxwcfyk72-bash-minimal-5.0.16/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)

and if the path of the binary executed would surface here then this would help - also the version would be nice to see. I did not see this genomeType error before - but that is about something else :o/

Update:

$ find /gnu/ -name STAR
/gnu/store/zd7gdv173x41rmjdgc7qr3ycasna28hd-star-2.7.3a/bin/STAR
/gnu/store/7bjj53q9lx87aw4ljs9b6wrs7qs4666v-star-2.7.3a/bin/STAR

just a hunch. But with that extra information in the logs, I would know for sure and could report with better confidence to the authors of pigx-rnaseq. Many thanks! Steffen

@alexdobin
Copy link
Owner

Hi Steffen,

good point, thanks! I should copy the entire command line into stdout, it would be useful.

Cheers
Alex

@alexdobin alexdobin merged commit d50175e into alexdobin:master Mar 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants