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

QCxMS: GFN1-xTB is never selected as method #511

Closed
hechth opened this issue Mar 21, 2024 · 4 comments · Fixed by #513
Closed

QCxMS: GFN1-xTB is never selected as method #511

hechth opened this issue Mar 21, 2024 · 4 comments · Fixed by #513
Assignees

Comments

@hechth
Copy link
Member

hechth commented Mar 21, 2024

We have to verify by running locally how the GFN1-xTB method is selected as this is never the case in the galaxy tool. So far, we have tried xtb and xtb1 as keywords but none of them triggers the correct method.

This has to be checked first locally, then identified which tag needs to be supplied and then the galaxy wrapper updated with this new tag and a accompanying test to avoid this happening again.

@wverastegui wverastegui self-assigned this Mar 22, 2024
@wverastegui
Copy link
Contributor

@hechth , @zargham-ahmad . The labels "xtb" for (GFN1-xTB) and "xtb2" for (GFN2-xTB) work as expected when locally run. The incorrect behavior must be on the Galaxy wrapper side.

@wverastegui wverastegui removed their assignment Mar 22, 2024
@zargham-ahmad
Copy link
Collaborator

@wverastegui did you check if there was a difference in the files generated by the local run and the files generated by Galaxy wrapper?

@wverastegui
Copy link
Contributor

wverastegui commented Mar 22, 2024

I haven't observed any differences so far.

@zargham-ahmad zargham-ahmad self-assigned this Mar 27, 2024
@zargham-ahmad
Copy link
Collaborator

the issue was with the leading spaces in the qcxms.in file.

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 a pull request may close this issue.

3 participants