-
Notifications
You must be signed in to change notification settings - Fork 0
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
zmq work manager fails against recent versions of zeromq #1
Comments
Documenting pyzmq API incompatibilities:
Making these two changes allows a bunch of the tests to successfully run, but I'm still unable to pass all of them. Currently hanging on the test |
Agreed. Especially given that I've waited on advances in numpy and h5py to do things I've wanted to do, and how very easy it is to use Anaconda to keep numpy, scipy, and h5py up-to-date, I think keeping current with major anaconda releases (those announced on their web site) is probably a decent benchmark. |
I've assigned this to myself, though PRs/patches are welcome. Until I get a chance to look seriously at this (within a month), any further details about which tests are failing (or hanging) would be helpful. |
@mczwier Any update on this? I saw you had a branch where you were working on this a while back, but it looks like you haven't had a chance to work on this recently. |
I've been using this on basically every cluster I have access to, and I haven't encountered a single bug as regards data I/O. About the only thing I've run into is it seems to need TCP, and it needs a longer timeout period than the old work manager. Adam |
@ajoshpratt when you say that you've been using "this", do you mean @mczwier's updated branch, or the current zeromq implementation that's in the master branch? |
Ah, yes, sorry. The updated branch on @mczwier's fork. Not the current implementation in master. |
Great. Are there plans to merge the refactored zmq work manager soon? Also, has there been any recent discussions of moving forward with some of the goals of packaging westpa using conda, setting up continuous integration, etc? |
Nothing in that vein, no. I've been prepping some significant changes on other parts (mostly the toolset) on the DEVELOPMENT branch. On that note, it would probably be useful for any interested parties to be involved in a discussion on the changes I've made there, if only so everyone is on board. They're all necessary; it's just a matter of whether the implementation is good, construction-wise. It might also be a good time to discuss the wwmgr fork with @mczwier. |
I recently cloned a fresh version of westpa and was testing it against Anaconda 2.1.0 (release before the most recent one), and noted that the zmq work manager wasn't an available option for
w_run
. It looks like there were some core changes to the pyzmq API which are breaking the work manager and the unit tests are failing, with:The text was updated successfully, but these errors were encountered: