-
Notifications
You must be signed in to change notification settings - Fork 653
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
Modin fails on a simple code snippet with ray==2.1.0
in development environment
#5208
Comments
It's unclear yet what (or which package) causing the problem. I would appreciate it if someone would also look into this. @modin-project/modin-core |
@dchigarev should we pin the version of Ray for now while we have someone look into the issue? |
sure, created a PR for this (#5209) |
Signed-off-by: Dmitry Chigarev <[email protected]>
There are a lot of things wrong with that environment:
Generally:
Of course things shouldn't break wherever possible, sorry for that (and we're looking into what's going on with grpcio 1.49.1 in this context). Still, I feel obliged to point out that your requirement constraints are somewhere between "actively hostile to the solver" and "shooting yourself in the foot". I've kept banging that drum in #3371 and on the feedstock, but I cannot seem to get that point across. It's one of the prime reasons I (personally) haven't spent much time with modin - it doesn't "play nice with others" from the POV of dependency constraints, which makes it very hard to use together with other projects (and their dependencies)1. Note that I'm not critizising temporary pins to unbreak CI, but with multiple several-year-old constraints, you're externalising a lot of costs onto the ecosystem (incl. other libraries, packagers and your users). Footnotes
|
@h-vetinari thank you for raising this point! I think a lot of what you said makes sense and can be addressed in a few PRs. I agree with you that pinning versions can cause dependency headaches. You raise a fair point w.r.t. pinning pandas to the patch level. We will take steps towards coming to a solution there! |
Signed-off-by: Myachev <[email protected]>
@h-vetinari I think it will be interesting for you to take a look #5270 (comment). In short, sometimes the installed |
Signed-off-by: Myachev <[email protected]>
Signed-off-by: Myachev <[email protected]>
Modin fails with the recently released ray 2.1.0 even for this simple code snippet:
Traceback
The log files that the error mentions seem to not contain any useful info:
dashboard_agent.log
python-core-driver0.log
The error occurs only on Linux and only in our development environment (environment-dev.yml). Fresh installed modin with ray 2.1.0 works fine:
The error also occurs in our CI where all of the non-windows jobs with ray fail. Example: https://github.com/modin-project/modin/actions/runs/3420080409/jobs/5697742798
p.s. the issue seems not to be related to the
redis
version requirements in our env recipe:modin/environment-dev.yml
Line 53 in c30ab4c
Both non-working and working environments use the same version of redis (3.5.1)
The text was updated successfully, but these errors were encountered: