Skip to content

Conversation

@sarahyurick
Copy link
Collaborator

@sarahyurick sarahyurick commented Jun 7, 2023

@charlesbluca
Copy link
Collaborator

Looking at that run, seems like the issue is that we're only pulling in Dask 2023.3.2? Going to check the conda environments of the GPU CI runs and see if something is constraining us to that specific version

@charlesbluca
Copy link
Collaborator

After creating the environment, it looks like this is because we're now beginning to pick up stable 23.06 packages that have pinned Dask:

$ mamba repoquery whoneeds dask
Using local repodata...

Loaded current active prefix: "/datasets/charlesb/micromamba/envs/dask-sql-gpuci-py39"

 Name      Version   Build                            Depends       Channel         
─────────────────────────────────────────────────────────────────────────────────────
 cuml      23.06.00a cuda11_py39_230607_g8cf3f92f6_98 dask 2023.3.2 rapidsai-nightly
 dask-cuda 23.06.00  py39_230607_gfd3ab2d_0           dask 2023.3.2 rapidsai        
 dask-cudf 23.06.00  cuda11_py39_230607_gf881d40c63_0 dask 2023.3.2 rapidsai        
 intake    0.7.0     pyhd8ed1ab_0                     dask >=1.0    conda-forge     
 raft-dask 23.06.00  cuda11_py39_230607_gc931b613_0   dask 2023.3.2 rapidsai   

This should be resolved with #1170, but we might be able to manually install Dask from main in your PR for now to test that the changes unblock

@codecov-commenter
Copy link

codecov-commenter commented Jun 7, 2023

Codecov Report

Merging #1174 (ade222e) into main (f0da15b) will decrease coverage by 0.27%.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main    #1174      +/-   ##
==========================================
- Coverage   81.42%   81.15%   -0.27%     
==========================================
  Files          78       78              
  Lines        4474     4474              
  Branches      817      817              
==========================================
- Hits         3643     3631      -12     
- Misses        650      654       +4     
- Partials      181      189       +8     

see 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sarahyurick sarahyurick closed this Jun 8, 2023
@sarahyurick sarahyurick deleted the dask_version branch June 8, 2023 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants