Skip to content

Fix conda forge package test workflow#601

Merged
popescu-v merged 4 commits into
mainfrom
fix-conda-forge-package-test-workflow
Jul 3, 2026
Merged

Fix conda forge package test workflow#601
popescu-v merged 4 commits into
mainfrom
fix-conda-forge-package-test-workflow

Conversation

@popescu-v

@popescu-v popescu-v commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Drop explicit khiops-core installation (it is installed through khiops).
Enable OpenMPI oversubscription (khiops-core 11.0.1 now uses OpenMPI on Conda as well).
Simplify the testing matrix: run Conda tests on one OS from each platform, with lowest / highest Python versions.


TODO Before Asking for a Review

  • Rebase your branch to the latest version of main (or main-v10)
  • Make sure all CI workflows are green
  • When adding a public feature/fix: Update the Unreleased section of CHANGELOG.md (no date)
  • Self-Review: Review "Files Changed" tab and fix any problems you find
  • API Docs (only if there are changes in docstrings, rst files or samples):
    • Check the docs build without warning: see the log of the API Docs workflow
    • Check that your changes render well in HTML: download the API Docs artifact and open index.html
    • If there are any problems it is faster to iterate by building locally the API Docs

popescu-v added 2 commits July 2, 2026 16:06
…rkflow

It is installed via the dependency specified in the recipe of the
`khiops` Conda package.
@popescu-v popescu-v requested a review from tramora July 2, 2026 14:36
@popescu-v popescu-v self-assigned this Jul 2, 2026
@popescu-v popescu-v force-pushed the fix-conda-forge-package-test-workflow branch from 2520d0e to d418f9f Compare July 2, 2026 14:39
@popescu-v popescu-v force-pushed the fix-conda-forge-package-test-workflow branch from d418f9f to bc3f49b Compare July 2, 2026 14:45
@popescu-v popescu-v requested a review from bruno-at-orange July 2, 2026 15:02

@bruno-at-orange bruno-at-orange left a comment

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.

LGTM

@tramora tramora left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved

@popescu-v popescu-v merged commit f3fa2be into main Jul 3, 2026
7 checks passed
@popescu-v popescu-v deleted the fix-conda-forge-package-test-workflow branch July 3, 2026 07:44
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.

3 participants