Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add pixi #412

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add pixi #412

wants to merge 6 commits into from

Conversation

maresb
Copy link
Contributor

@maresb maresb commented Sep 16, 2024 •
edited by github-actions bot
Loading

With this you can

pixi run init-jupyter-kernel
pixi shell
jupyter lab

Side note:

I'm working on a guide for migrating to pixi, and a tool to do it automatically via conda-lock.

The guide is here: conda/conda-lock#708: (preview)

Note that because things are unreleased, to follow the guide you need to install conda-lock via

pipx install --force git+https://github.com/conda/conda-lock@main

📚 Documentation preview 📚: https://causalpy--412.org.readthedocs.build/en/412/

@maresb
Ignore pixi cache directory
3b24297
@maresb
Add pixi.toml configuration
78feb4d 
Generated with:

```bash
conda-lock render-lock-spec --kind=pixi.toml --stdout \
    --file=pyproject.toml \
    --channel=conda-forge \
    --pixi-project-name=CausalPy \
    --editable causalpy=. \
    > pixi.toml
```
@maresb
Fix pixi.toml
dde6382 
Some PyPI packages aren't available on conda-forge
@maresb
Add Jupyter feature/task
d7332dc
@maresb
Add linux-aarch64 platform to pixi
089e89f 
This is useful for Docker on Apple Silicon
@maresb
Add pixi.lock lock file
c12df48
@codecov Codecov
Copy link

codecov bot commented Sep 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.40%. Comparing base (66c2623) to head (c12df48).
Report is 6 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   94.35%   94.40%   +0.05%     
==========================================
  Files          30       31       +1     
  Lines        1967     1985      +18     
==========================================
+ Hits         1856     1874      +18     
  Misses        111      111

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@drbenvincent drbenvincent self-requested a review September 17, 2024 07:48
@maresb
Copy link
Contributor Author

maresb commented Sep 17, 2024

I should add @aseyboldt's CUDA stuff here too. But that shouldn't block us from reviewing the rest of this.

@maresb maresb mentioned this pull request Sep 19, 2024
Merged
13 tasks
juanitorduz
juanitorduz approved these changes Oct 18, 2024
View reviewed changes
Copy link
Collaborator

@juanitorduz juanitorduz left a comment

Choose a reason for hiding this comment

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

I love Pixi!

drbenvincent
drbenvincent requested changes Oct 18, 2024
View reviewed changes
Copy link
Collaborator

@drbenvincent drbenvincent left a comment

Choose a reason for hiding this comment

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

Think this is great, though not had a chance to play with it. Think it could be worth adding some instructions to CONTRIBUTING.md for those who are new to pixi? Or is the idea to build that out in the future?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drbenvincent drbenvincent drbenvincent requested changes

@juanitorduz juanitorduz juanitorduz approved these changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maresb @drbenvincent @juanitorduz