Skip to content

DEV/CI: pixi-build setup & Pixi for CI and dev docs#314

Open
lucascolley wants to merge 8 commits into
finch-tensor:mainfrom
lucascolley:pixi
Open

DEV/CI: pixi-build setup & Pixi for CI and dev docs#314
lucascolley wants to merge 8 commits into
finch-tensor:mainfrom
lucascolley:pixi

Conversation

@lucascolley

Copy link
Copy Markdown

No description provided.

@lucascolley

Copy link
Copy Markdown
Author

hey @willow-ahrens, hope this can be useful

Comment thread .github/workflows/ci.yml Outdated
@mtsokol mtsokol self-requested a review February 5, 2026 22:05
Comment thread pixi.toml Outdated
Comment on lines +25 to +29
[package.run-dependencies]
numpy = "*"
numba = "*"
donfig = "*"
lark = "*"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

per conda-forge/staged-recipes#32124 it looks like we may want to add a run dependency on a c-compiler for people working in particularly empty global environments.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We have been thinking that the runtime c-compiler dep would be an optional backend for users. The main backend is Numba for conda installs, but I think that our CI tests require the C compiler.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd like to leave the C compiler as an optional requirement to reduce the dependencies.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I'd like to leave the C compiler as an optional requirement to reduce the dependencies.

sounds good! So we need to make the C compiler tests opt-in or automatically skipped when there isn't one available?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yes.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

So we need to make the C compiler tests opt-in or automatically skipped when there isn't one available?

yes.

hey @willow-ahrens @mtsokol ! Just wondering if there is any update on this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

(if there is some actionable work I can try to help out!)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

gentle bump

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the reminder, I'm still working on this! We're reorganizing the github org and projects a little bit lately. We're resolved now to switch to pixi, make dep on GCC optional, and also add an optional julia dep.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I may need a little help getting the gcc dep to work, but I'll reach out if it's an issue.

@lucascolley

Copy link
Copy Markdown
Author

hopefully that fixes CI

@codecov

codecov Bot commented Feb 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mtsokol

mtsokol commented Feb 25, 2026

Copy link
Copy Markdown
Member

I'll take a look at Windows failures - win job runs in main apparently.

@willow-ahrens willow-ahrens linked an issue Jun 30, 2026 that may be closed by this pull request
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.

Switch to Pixi, add optional gcc backends

3 participants