Remove dask and distributed dependencies #2691
Conversation
multimodal/setup.py
Outdated
| @@ -55,6 +55,7 @@ | |||
| "openmim>0.1.5,<=0.2.1", | |||
| "defusedxml>=0.7.1,<=0.7.1", | |||
| "albumentations>=1.1.0,<=1.2.0", | |||
| "jinja2>=3.0.3,<3.2", | |||
There was a problem hiding this comment.
Where do we use jinja2?
There was a problem hiding this comment.
There was a problem hiding this comment.
You can refer to the CI error prior to me adding it in this PR: https://github.com/autogluon/autogluon/actions/runs/3914783191/jobs/6692268378#step:4:994
There was a problem hiding this comment.
This is for supporting prompt source. I’m okay with removing it for now.
There was a problem hiding this comment.
@sxjscience if you'd like to remove jinja usage, you'll need to open a separate follow-up PR. I believe jinja was a dependency of dask/distributed, which is why we didn't error in the past, but with dask/distributed removed, now we need to explicitly add it as a dependency to AutoMM
Innixma
force-pushed
the
try_remove_dask
branch
2 times, most recently
from
047924e
to
4824165
Compare
Innixma
force-pushed
the
try_remove_dask
branch
from
4824165
to
b63d6f9
Compare
|
Job PR-2691-63ac256 is done. |
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.