Skip to content
This repository has been archived by the owner on May 19, 2023. It is now read-only.

Update conda environment name for CI #436

Merged
merged 2 commits into from
Jul 8, 2021

Conversation

ajschmidt8
Copy link
Member

The gdf conda environment has been replaced with the rapids environment. A symlink was put in place for gdf to continue to work, but the symlink will be removed in the near future. This PR updates all scripts to use the rapids environment name.

@ajschmidt8 ajschmidt8 requested a review from a team as a code owner July 8, 2021 17:54
@ajschmidt8 ajschmidt8 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 8, 2021
@github-actions github-actions bot added the gpuCI label Jul 8, 2021
@github-actions github-actions bot added the conda label Jul 8, 2021
@ajschmidt8
Copy link
Member Author

@efajardo-nv and/or @BartleyR, can you review/test the changes in a225092? I believe that was causing the CI issues seen in my previous commit (log link). CI is running with Python 3.8, but gwerbin only has a 3.7 version of seqeval on Anaconda. Therefore conda was installing the seqeval 0.0.12 version from conda-forge, which was downgrading arrow to 1.0.1, which we no longer use as of rapidsai/cudf#7495. From my local testing, upgrading to seqeval 1.2.2 seems to prevent arrow from being downgraded.

Here's the changelog related to seqeval 1.2.2 for reference: https://github.com/chakki-works/seqeval/releases/tag/v1.2.2

@ajschmidt8
Copy link
Member Author

Actually, upon further inspection, seqeval 1.2.2 is what's in the Docker images by default right now (see screenshot below). And the cybert_example_training.ipynb notebook seems to run fine there. So I think this change should be fine.

image

@ajschmidt8
Copy link
Member Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 12ec6c0 into rapidsai:branch-21.08 Jul 8, 2021
@ajschmidt8 ajschmidt8 deleted the update-conda-env branch July 9, 2021 13:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
conda gpuCI improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants