Skip to content

Rebuild for pytorch 2.5#20

Closed
regro-cf-autotick-bot wants to merge 7 commits into
conda-forge:mainfrom
regro-cf-autotick-bot:rebuild-pytorch25-0-1_h679dff
Closed

Rebuild for pytorch 2.5#20
regro-cf-autotick-bot wants to merge 7 commits into
conda-forge:mainfrom
regro-cf-autotick-bot:rebuild-pytorch25-0-1_h679dff

Conversation

@regro-cf-autotick-bot

Copy link
Copy Markdown
Contributor

This PR has been triggered in an effort to update pytorch25.

Notes and instructions for merging this PR:

  1. Please merge the PR only after the tests have passed.
  2. Feel free to push to the bot's branch to update this PR if needed.

Please note that if you close this PR we presume that the feedstock has been rebuilt, so if you are going to perform the rebuild yourself don't close this PR until the your rebuild has been merged.


If this PR was opened in error or needs to be updated please add the bot-rerun label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase @conda-forge-admin, please rerun bot in a PR comment to have the conda-forge-admin add it for you.

This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/11695311053 - please use this URL for debugging.

@conda-forge-admin

conda-forge-admin commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

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 (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12943976494. Examine the logs at this URL for more detail.

@hmaarrfk

Copy link
Copy Markdown

@conda-forge-admin please rerender

Comment thread recipe/meta.yaml

package:
name: {{ pname|lower }}
name: aihwkit

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unlike pypi packages, we have the ability to have different variants of our packages with hardware capabilities.

To avoid users having to create different environment.yml files for cuda/cpu, we tend to keep the name of the hardware accelerated packages the same.

I only created the pytorch-cpu and pytorch-gpu since at the time, I was following the upstream pytorch package names.

To keep things "similar" for your users, i've created the aihwkit-cpu and aihwkit-gpu packages in addition to your naked name aihwkit.

Recipe can now depend on aihwkit unconditionally.

conda-forge-webservices[bot] and others added 2 commits November 24, 2024 04:18
Comment thread recipe/meta.yaml
{% set name = "aihwkit" %}
{% set version = "0.9.2" %}
{% set number = 0 %}
{% set build = 1 %}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using {{ number }} messes with the bot's ability to bump it on PRs.

You should use {{ build }}

Comment thread recipe/meta.yaml
- cuda-version {{ cuda_compiler_version }} # [cuda_compiler_version != "None"]
- libcublas-dev # [(cuda_compiler_version or "").startswith("12")]
- libcurand-dev # [(cuda_compiler_version or "").startswith("12")]
- mkl-devel

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mkl-devel should include mkl and mkl-include (as well as blas I believe) and provide you with the correct run exports.

@hmaarrfk

Copy link
Copy Markdown

@conda-forge-admin please rerender

@h-vetinari

Copy link
Copy Markdown
Member

@conda-forge-admin, please rerender

@h-vetinari

Copy link
Copy Markdown
Member

AFAICT, this package still uses an old pip argument that got removed

no such option: --install-option

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.

5 participants