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

deepTools-3.5.4 :: estimateScaleFactor :: typo #1286

Closed
2 tasks
EricDeveaud opened this issue Jan 16, 2024 · 1 comment
Closed
2 tasks

deepTools-3.5.4 :: estimateScaleFactor :: typo #1286

EricDeveaud opened this issue Jan 16, 2024 · 1 comment

Comments

@EricDeveaud
Copy link

EricDeveaud commented Jan 16, 2024

Welcome to deepTools GitHub repository! Before opening the issue please check
that the following requirements are met :

  • Paste your deepTools version (deeptools --version) and your python
    version (python --version) below.
deepTools/3.5.4 > deeptools --version
deeptools 3.5.4
deepTools/3.5.4 > /opt/gensoft/exe/deepTools/3.5.4/venv/bin/python3 --version
Python 3.8.1
  • Paste the full deepTools command that produces the issue below
deepTools/3.5.4 > estimateScaleFactor --bamfiles ../../datas/deepTools/phiX.bam ../../datas/deepTools/phiX.bam --numberOfSamples 1000
Traceback (most recent call last):
  File "/opt/gensoft/exe/deepTools/3.5.4/bin/estimateScaleFactor", line 8, in <module>
    sys.exit(main())
  File "/opt/gensoft/exe/deepTools/3.5.4/venv/lib/python3.8/site-packages/deeptools/estimateScaleFactor.py", line 105, in main
    args = parser.parse_args(args)
NameError: name 'parser' is not defined

deepTools 3.5.4 installed from release tag: https://github.com/deeptools/deepTools/archive/3.5.4.tar.gz

seems to me that as ParseArguments returns the whole arguments already parsed.
line 105 should be just args = parseArguments(args)

regards

Eric

@WardDeb WardDeb mentioned this issue Jan 17, 2024
4 tasks
@WardDeb
Copy link
Member

WardDeb commented Jan 17, 2024

Thanks for the report ! you're right. Fix has been merged into develop, there's a new release coming soon.

Kind regards,

WardDeb

@WardDeb WardDeb closed this as completed Jan 17, 2024
@WardDeb WardDeb mentioned this issue Mar 8, 2024
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

No branches or pull requests

2 participants