-
Notifications
You must be signed in to change notification settings - Fork 11
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
make_queue_pairs removed? #96
Comments
Thanks for reaching out! I did remove that function when refactoring Colmena and forgot to update the example. We now use a single object to define queues and offer a few different implementations of them. The above example has been replaced by:
|
I'll go fix that in the demo |
Fantastic! Thank you - will try this out right now. |
Do you have any advice for debugging (this is the first time I'm using these libraries). It's hanging on this line. For reference I am updating the example from this notebook which I'm not sure was run because it has the wrong import path. I'm running redis inside of the same container where I'm testing this, so it should be on localhost! Complete instructions (so far) are in the README. Thanks for the help! Sorry for being a noob. 😆 |
Sorry for the hassle and missing your question. Not a noob question at all! (And it wouldn't be a problem even if it was!) I ended up having a hang in a similar spot, due to the task failing. Check the "colmena.log" to see if there was an error. I just updated the demo app and changed a few things about the environment to |
I don't see an error, but I see these two lines a gazillion times:
And before those repetitions, it seems mostly ok?
|
Yes, those lines are fine. I'll going to push a version of colmena to PyPI that I hopes fixes that excessive logging |
Thank you! I reverted some of the version pinning and got the first example running again - when it finishes I'll try this new approach with each of redis and the pipe variant. |
Mind if I close this issue? The problems now are with the example, not Colmena, from my understanding. |
Definitely - we resolved this one! Thanks for the help, closing. |
Hi! I'm using a script from 8 months ago with:
But
colmena.queue.redis
has no function called "make_queue_pairs". What is the replacement for this now? I've tried installing older versions but then other bugs emerge.Related: ExaWorks/molecular-design-parsl-demo#2
Thanks!
The text was updated successfully, but these errors were encountered: