Skip to content

Test: CPU vs GPU workaround#332

Draft
h-vetinari wants to merge 4 commits intoconda-forge:mainfrom
h-vetinari:cpu_vs_gpu_isuru
Draft

Test: CPU vs GPU workaround#332
h-vetinari wants to merge 4 commits intoconda-forge:mainfrom
h-vetinari:cpu_vs_gpu_isuru

Conversation

@h-vetinari
Copy link
Copy Markdown
Member

@h-vetinari h-vetinari commented Jan 26, 2025

Test whether 200 characters are short enough for the windows server; uses a conda-smithy patch as proposed here

Includes #318, but that's kinda irrelevant

@conda-forge-admin
Copy link
Copy Markdown
Contributor

conda-forge-admin commented Jan 26, 2025

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.
  • ℹ️ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.

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

This was referenced Jan 26, 2025
Merged
Merged
Closed
@Tobias-Fischer
Copy link
Copy Markdown
Contributor

Tobias-Fischer commented Jun 4, 2025

I think this can probably be closed @h-vetinari?

@h-vetinari
Copy link
Copy Markdown
Member Author

h-vetinari commented Jun 4, 2025

I think this can probably be closed @h-vetinari?

After demanding the revert of conda-forge/conda-forge-pinning-feedstock@ff043bb, Isuru promised to help fix this issue (c.f. conda-forge/conda-forge-pinning-feedstock#6967). It's still unresolved.

@conda-forge-admin
Copy link
Copy Markdown
Contributor

conda-forge-admin commented Dec 11, 2025 •
edited
Loading

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 (recipe/meta.yaml) and found some lint.

Here's what I've got...

For recipe/meta.yaml:

  • ❌ The recipe is not parsable by any of the known recipe parsers (['conda-forge-tick (the bot)', 'conda-recipe-manager', 'conda-souschef (grayskull)']). Please check the logs for more information and ensure your recipe can be parsed.

For recipe/meta.yaml:

  • ℹ️ The magma output has been superseded by libmagma-devel.
  • ℹ️ The recipe is not parsable by parser conda-forge-tick (the bot). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.
  • ℹ️ 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.
  • ℹ️ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.

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

This was referenced Dec 11, 2025
Merged
Merged
@conda-forge-admin
Copy link
Copy Markdown
Contributor

conda-forge-admin commented Dec 11, 2025 •
edited
Loading

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 magma output has been superseded by libmagma-devel.
  • ℹ️ 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.
  • ℹ️ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.

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

@h-vetinari h-vetinari mentioned this pull request Dec 14, 2025
Closed
h-vetinari and others added 3 commits December 29, 2025 13:01
@h-vetinari @isuruf
build non-CUDA builds on CPU agents
6486533
@isuruf
MNT: Re-rendered with conda-smithy 3.54.2.dev1+gec879dc6 and conda-fo…
0b36150 
…rge-pinning 2025.12.29.20.21.59

Other tools:
- conda-build 25.9.0
- rattler-build 0.47.0
- rattler-build-conda-compat 1.4.5
@isuruf
remove used
2876fc7
carterbox
carterbox reviewed Jan 14, 2026
View reviewed changes
Comment thread recipe/meta.yaml
Comment on lines +69 to +72
# CUDA builds on GPU agents, non-CUDA builds on CPU agents; see above
{% if (cuda_compiler_version == "None" and "gpu" in github_actions_labels) or (cuda_compiler_version != "None" and "cpu" in github_actions_labels) %}
skip: true
{% endif %}
Copy link
Copy Markdown
Member

@carterbox carterbox Jan 14, 2026

Choose a reason for hiding this comment

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

This should work as soon as conda-smithy > 3.54.1 is released!

@h-vetinari h-vetinari mentioned this pull request Jan 19, 2026
Merged
5 tasks
@h-vetinari h-vetinari mentioned this pull request Jan 26, 2026
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Tobias-Fischer Tobias-Fischer Awaiting requested review from Tobias-Fischer Tobias-Fischer will be requested when the pull request is marked ready for review Tobias-Fischer is a code owner

@baszalmstra baszalmstra Awaiting requested review from baszalmstra baszalmstra will be requested when the pull request is marked ready for review baszalmstra is a code owner

@beckermr beckermr Awaiting requested review from beckermr beckermr will be requested when the pull request is marked ready for review beckermr is a code owner

@benjaminrwilson benjaminrwilson Awaiting requested review from benjaminrwilson benjaminrwilson will be requested when the pull request is marked ready for review benjaminrwilson is a code owner

@hmaarrfk hmaarrfk Awaiting requested review from hmaarrfk hmaarrfk will be requested when the pull request is marked ready for review hmaarrfk is a code owner

@jeongseok-meta jeongseok-meta Awaiting requested review from jeongseok-meta jeongseok-meta will be requested when the pull request is marked ready for review jeongseok-meta is a code owner

@mgorny mgorny Awaiting requested review from mgorny mgorny will be requested when the pull request is marked ready for review mgorny is a code owner

@sodre sodre Awaiting requested review from sodre sodre will be requested when the pull request is marked ready for review sodre is a code owner

1 more reviewer

@carterbox carterbox carterbox left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@h-vetinari @conda-forge-admin @Tobias-Fischer @carterbox @isuruf