Skip to content

[WIP] I am trying to remove the restriction for not using functions in the snakefile output block. However the added test case fail because parsing of callable outputs is not implemented correctly. Fix this.#1

Closed
Copilot wants to merge 1 commit intofunction_outputfrom
copilot/fix-02fe1718-33dd-4d00-b572-9e8af260df80

Conversation

Copy link

Copilot AI commented Aug 21, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original description:

I am trying to remove the restriction for not using functions in the snakefile output block. However the added test case fail because parsing of callable outputs is not implemented correctly. Fix this.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI requested a review from ckronlage August 21, 2025 16:54
@ckronlage ckronlage closed this Aug 22, 2025
fgvieira pushed a commit that referenced this pull request Feb 27, 2026
Pixi install github action is failing with "failed to parse pypi name
mapping" errors likely due to rate limiting when 30+ jobs are kicked off
nearly simultaneously

I tested this fix on snakemake#3820 since the tests kept failing due to the
`pixi` install action failing. After committing this change, [the
actions ran
successfully](https://github.com/snakemake/snakemake/actions/runs/20684893321).

In [this failing
run's](https://github.com/snakemake/snakemake/actions/runs/20682879051/job/59383597583#step:3:3261)
debug logs we see:
```
pixi install -e py311
[...]
   WARN resolve_conda{group=py313 platform=win-64}: reqwest_retry::middleware: Retry attempt #1. Sleeping 1.225245051s before the next attempt
  Error:   × failed to parse pypi name mapping
    ├─▶ error decoding response body
    ╰─▶ expected value at line 1 column 1
```
This warning is repeated many times until finally pixi stops retrying -
this is what suggested to me that some sort of rate limit was the issue.


One downside is that this does make the CI take a bit longer to run. We
could consider using the `cache` feature of the pixi action. And turning
up the max-parallel, or reducing the number of test-groups


### QC
<!-- Make sure that you can tick the boxes below. -->

* [ ] The PR contains a test case for the changes or the changes are
already covered by an existing test case.
* [ ] The documentation (`docs/`) is updated to reflect the changes or
this is not necessary (e.g. if the change does neither modify the
language nor the behavior or functionalities of Snakemake).



<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
* Updated development toolchain dependencies for improved build and test
infrastructure.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
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.

2 participants