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

pydpiper 2.0.10 provides inappropriate memory specification to qbatch for SGE #350

Open
gdevenyi opened this issue Nov 30, 2017 · 5 comments

Comments

@gdevenyi
Copy link

Users of qbatch are required to format their memory specification according to how the queuing system accepts it, qbatch does no testing.

"GB" is not an accepted input format for SGE.

@bcdarwin
Copy link
Member

bcdarwin commented Dec 1, 2017

Interesting ... that means qbatch actually isn't an abstraction over the queueing system. I guess I'll add some way to format memory specs to Pydpiper, then.

Sorry for the inconvenience ... the whole reason for the qbatch switch was that we moved from SGE to slurm (so don't have an SGE system for testing anymore).

@gdevenyi
Copy link
Author

gdevenyi commented Dec 1, 2017

I can add it I guess. Pypy needs a new release anyways

@gdevenyi
Copy link
Author

gdevenyi commented Dec 1, 2017

Time for Travis testing?

@gdevenyi
Copy link
Author

Fun fact: PBS documentation doesn't mention capitalized memory submission options as being valid :)

@gdevenyi
Copy link
Author

Helping someone get pydpiper working on their cluster and bitten by this again.

I have the interface idea listed at CoBrALab/qbatch#145, contributions are welcome :)

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