Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[NO MRG] Test CI #692

Closed
wants to merge 1 commit into from
Closed

[NO MRG] Test CI #692

wants to merge 1 commit into from

Conversation

jakirkham
Copy link
Member

No description provided.

@jakirkham jakirkham mentioned this pull request Jul 18, 2024
Merged
@jakirkham
Copy link
Member Author

One of the jobs had trouble during building finding the rapids=24.08 package. Likely a network issue as the packages appear to be there and we haven't had issues in other builds. Restarted all jobs to have a clean build

@jameslamb
Copy link
Member

thanks @jakirkham .

I just remembered I have #688, want to use that to continue testing CI? It isn't adding CUDA 12.5 support so shouldn't be blocked by cuspatial, and it at least should eventually be merged (to fix some very minor stylistic warnings).

I restarted CI there a few minutes ago.

@jakirkham
Copy link
Member Author

We can certainly do that

Looks like we are down to the final 3 jobs here. So may just wait and see what happens

@jameslamb
Copy link
Member

Looks like tests did pass here and in #688! I'm going to merge #688, and I think we can close this.

@jameslamb jameslamb closed this Jul 18, 2024
@jakirkham jakirkham deleted the tst_ci branch July 18, 2024 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jakirkham @jameslamb