-
-
Notifications
You must be signed in to change notification settings - Fork 151
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
Update Dask version to 2022.2.1 #205
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @jsignell. This should be good to go once 2022.2.1
lands on conda-forge
closing in favor of #206 |
@jacobtomlinson it looks like the 3.8 base build is failing. I'm not quite sure how to go about debugging. |
The error says |
Yeah I tested moving the python install up into the top line and it seems to work better locally. Can you see if that change looks sane to you? |
@@ -10,10 +10,8 @@ ENV PATH /opt/conda/bin:$PATH | |||
ENV PYTHON_VERSION=${python} | |||
ENV DASK_VERSION=${release} | |||
|
|||
RUN conda install --yes nomkl cytoolz cmake \ | |||
&& conda install --yes mamba \ | |||
RUN conda install --yes -c conda-forge nomkl cytoolz cmake mamba python=${PYTHON_VERSION} \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved the python install up, and added conda-forge
to this line to make sure we don't have different channels.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm that doesn't seem to have quite done it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updating the base image version did the trick -- thanks @jsignell!
Thanks @jrbourbeau!! |
yep! I mildly tried to fix the bot, but I think that would require knowing the previous version which I guess we could hard code or pull from main? I did not get very far... |
Yeah I have as well, but I think there was something I wasn't getting right about how the regex got applied |
Thanks for handling this folks! |
The last package is building on conda-forge shortly, so I figured I'd put this up.