-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
v1.8 -> v1.9 #44
v1.8 -> v1.9 #44
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipe:
For recipe:
|
@conda-forge-admin please rerender |
@isuruf, it appears that some of the Windows builds are immediately failing. Do you think we can just skip these? |
@hmaarrfk, would you mind taking a look at this when you get a chance as well? |
Hi! This is the friendly automated conda-forge-linting service. I was trying to look for recipes to lint for you, but it appears we have a merge conflict. Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug. |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipe:
For recipe:
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipe:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge/core, what is the best practice here for circumventing the Azure CI timeout? Is the only option to build locally? |
Currently yes. We have been buliding these packages manually on personally hardware. Rather tedious but dooable for linux. Did you get the recipe updated? Seems like things are passing. Encouraging! |
Ah, I see. Yes, the recipe is updated and seems to be working. I wonder if we could limit gpu architectures that this is being built against to try and get within the timeout limit. |
I wonder if we should wait for the MKL migration as well. I think the bot is going to be doing an other round and starting a build soon. Do you want me to test building a CUDA build? |
@hmaarrfk, a CUDA build would be much appreciated! I wonder if there's a way to split up the compute architecture builds into separate jobs (to try to avoid the CI timeout)? |
I might not have time to start one until Sunday Afternoon. For what its worth, you should be able to test local hardware with |
My machiens are just currently in use. |
I'm kinda happy with this as is now. A GPU build Was successful. @isuruf do we want to build them all here before merging? |
@isuruf thank you for the review. I started a build tracker. For those that want to help chose a build, and run the command:
and fill in the table with your name and start time (just so that we can get a feeling of the expected end date). I started 4 at time on a largeish machine i have. I hope that the machine can handle it. |
nvm, on my computer, the disk is full..... managing the space taken by docker is no fun.... |
Please wait until the review is over and this Pr is merged |
ok, will do. |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipe:
|
@conda-forge-admin, rerender |
…nda-forge-pinning 2021.07.25.11.01.12
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)