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

Implement a wrapper to run jug with gridmap #23

Closed
wants to merge 3 commits into from

Conversation

andsor
Copy link
Contributor

@andsor andsor commented Jul 29, 2015

A light-weight wrapper to run jug with gridmap on a Grid Engine cluster

Get the best of both worlds of gridmap and jug

This implements jug.grid_jug to run jug jobs with all the comfortable features of gridmap.

As I am not sure whether this belongs into the gridmap or the jug package, I create a pull request for both to foster discussion between the two projects.
The file jug/grid.py is identical with the file gridmap/jug.py .

@andsor
Copy link
Contributor Author

andsor commented Jul 29, 2015

Simultaneous PR: pygridtools/gridmap#58

@luispedro
Copy link
Owner

Thanks for this.

I think this could be a good idea (although I hadn't heard of gridmap until now). I would just like to see a bit more documentation (for myself) before I can accept the pull request.

(There is a minor travis failure, which also needs to be fixed, but I think that one should be a trivial Python2-vs-Python3 thing)

@andsor
Copy link
Contributor Author

andsor commented Aug 14, 2015

Check, I am on it now

A light-weight wrapper to run jug with gridmap on a Grid Engine cluster
@andsor
Copy link
Contributor Author

andsor commented Aug 14, 2015

Rebased on top of current master

@andsor
Copy link
Contributor Author

andsor commented Aug 14, 2015

Fixed Travis

@andsor
Copy link
Contributor Author

andsor commented Aug 14, 2015

Ola, thanks for considering this!

@andsor
Copy link
Contributor Author

andsor commented Aug 17, 2015

I'm sorry it seems I cannot get it to work within the jug package on a cluster due to unpickling error (relative imports). I better move this into a small addon package.

@andsor
Copy link
Contributor Author

andsor commented Aug 17, 2015

Ok I put into this repository https://github.com/andsor/gridjug and got it up and running on the cluster. Closing this now.

@andsor andsor closed this Aug 17, 2015
@luispedro
Copy link
Owner

I suggest you announce your gridjug project at https://groups.google.com/group/jug-users/

It's a small group, but it'll get archived.

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.

None yet

2 participants