Skip to content

DEVOPS-693: pixi all the way, including for doc - #34

Open
sebhmg wants to merge 23 commits into
developfrom
DEVOPS-693
Open

DEVOPS-693: pixi all the way, including for doc#34
sebhmg wants to merge 23 commits into
developfrom
DEVOPS-693

Conversation

@sebhmg

@sebhmgsebhmg commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

DEVOPS-693 - migrate from conda-lock to pixi

  • fix test for remove the jinja dependency
  • pixi task for build the doc, used in RTD
  • tweak RTD configuration
  • replace install_in_analyst bat by pixi task

@github-actionsgithub-actionsBot changed the title [DEVOPS-693] pixi all the way, including for docDEVOPS-693: pixi all the way, including for docSep 16, 2025
- fix test for remove the jinja dependency
- pixi task for build the doc, used in RTD
- tweak RTD configuration
- replace install_in_analyst bat by pixi task
- use pixi to run pylint from pre-commit
@sebhmg

Copy link
Copy Markdown
ContributorAuthor

doc is building but has warnings (I set the flag to report them as errors...)
See https://app.readthedocs.com/projects/mirageoscience-grid-apps/builds/3362588/

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates the project from conda-lock to pixi for dependency management. It removes jinja2 dependency from tests, configures pixi environments, and adds new documentation build tasks.

  • Removes all conda-lock files and replaces dependency management with pixi configuration
  • Removes jinja2 templating from test files and simplifies recipe version reading
  • Updates CI/CD workflows and documentation configuration to use pixi

Reviewed Changes

Copilot reviewed 29 out of 34 changed files in this pull request and generated 2 comments.

Show a summary per file
FileDescription
tests/version_test.pyRemoves jinja2 dependency and simplifies recipe.yaml parsing
pyproject.tomlMoves dependencies from poetry to standard project section and updates keywords
pixi.tomlAdds comprehensive pixi configuration with environments and tasks
install_in_analyst.batRemoves conda-based installation script
.readthedocs.ymlUpdates RTD config to use pixi for documentation builds
.github/workflows/python_analysis.ymlUpdates CI workflows to use pixi package manager

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment threadpyproject.toml
Comment on lines +38 to +43
"geoapps-utils >= 0.6.0a, == 0.6.*",
"geoh5py >= 0.12.0a1, == 0.12.*",
# other direct dependencies
"discretize == 0.11.*",
"numpy == 1.26.*",
"pydantic >= 2.5.2, == 2.*",

CopilotAISep 16, 2025

Copy link

Choose a reason for hiding this comment

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

The dependency specification combines both minimum version and exact version constraints (e.g., >= 0.6.0a, == 0.6.*). This pattern is redundant since the exact version constraint already implies the minimum. Consider using only the exact version constraint format.

Suggested change
"geoapps-utils >= 0.6.0a, == 0.6.*",
"geoh5py >= 0.12.0a1, == 0.12.*",
# other direct dependencies
"discretize == 0.11.*",
"numpy == 1.26.*",
"pydantic >= 2.5.2, == 2.*",
"geoapps-utils == 0.6.*",
"geoh5py == 0.12.*",
# other direct dependencies
"discretize == 0.11.*",
"numpy == 1.26.*",
"pydantic == 2.*",

Copilot uses AI. Check for mistakes.
Comment threadpixi.toml
cmd = "sphinx-build -T -nW -b {{builder}} docs/source {{outputdir}}/{{builder}}"

[tasks.show-pip-source]
args = [{ arg = "packages", "default" = "geoh5py geoapps-utils" }]

CopilotAISep 16, 2025

Copy link

Choose a reason for hiding this comment

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

The key "default" should be default without quotes for consistency with other argument definitions in the file.

Suggested change
args = [{ arg = "packages", "default" = "geoh5py geoapps-utils" }]
args = [{ arg = "packages", default = "geoh5py geoapps-utils" }]

Copilot uses AI. Check for mistakes.
domfournierand others added 21 commits April 21, 2026 09:44
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: domfournier <dominiquef@mirageoscience.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
(cherry picked from commit c534720)
# Conflicts:
#	.github/workflows/python_analysis.yml
#	.gitignore
#	deps-lock-config.yaml
#	docs/environment.yml
#	environments/env-python-3.13.yml
#	environments/env-python-3.14.yml
#	environments/py-3.12-linux-64-dev.conda.lock.yml
#	environments/py-3.12-linux-64.conda.lock.yml
#	environments/py-3.12-win-64-dev.conda.lock.yml
#	environments/py-3.12-win-64.conda.lock.yml
#	py-3.12.conda-lock.yml
#	pyproject.toml
#	tests/version_test.py
Sign up for freeto 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.

4 participants

@sebhmg@domfournier@benk-mira